Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

For VDDK help text - replace the example with a format #1389

Merged

Conversation

sgratch
Copy link
Collaborator

@sgratch sgratch commented Nov 19, 2024

Reference: https://issues.redhat.com/browse/MTV-1552

To avoid users from using the URL example which was provided for the Vsphere's VDDK help text field (.....for example: quay.io/kubev2v/vddk:latest ) ,
replace the help text to use a format suggestion instead (...in the format of <registry_route_or_server_path>/vddk:<tag> ..)

This is relevant for the create, edit, tooltip fields for all validaiton statuses (error, success, default).

Screenshots

Before

Screenshot from 2024-11-19 21-09-33
Screenshot from 2024-11-19 20-27-12
Screenshot from 2024-11-19 20-26-48

After

Screenshot from 2024-11-19 21-12-45
Screenshot from 2024-11-19 21-11-09
Screenshot from 2024-11-19 21-11-40

Reference: https://issues.redhat.com/browse/MTV-1552

To avoid users from using the URL example which was provided for the Vsphere's
VDDK help text field, replace it with a format suggestion.

This is relevant for the create, edit, tooltip fields for all validaiton
statuses (error, success, default).

Signed-off-by: Sharon Gratch <[email protected]>
@sgratch sgratch requested a review from yaacov November 19, 2024 19:13
@sgratch sgratch added the bug Categorizes issue or PR as related to a bug. label Nov 19, 2024
@sgratch sgratch added this to the 2.8.0 milestone Nov 19, 2024
@sgratch
Copy link
Collaborator Author

sgratch commented Nov 19, 2024

cc: @RichardHoch
Please review the warning message displayed in the screenshots within main comment:
#1389 (comment)

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 36.16%. Comparing base (13484d0) to head (621f86a).
Report is 122 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1389      +/-   ##
==========================================
- Coverage   36.81%   36.16%   -0.65%     
==========================================
  Files         158      158              
  Lines        2548     2577      +29     
  Branches      599      616      +17     
==========================================
- Hits          938      932       -6     
- Misses       1428     1450      +22     
- Partials      182      195      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@metalice
Copy link
Collaborator

/lgtm

@sgratch sgratch merged commit 7036ecd into kubev2v:main Nov 21, 2024
8 of 9 checks passed
@sgratch sgratch deleted the replace-vddk-help-msg-example-with-format branch November 21, 2024 11:07
@sgratch sgratch modified the milestones: 2.8.0, 2.7.6 Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants