Bug 1541393
Summary: | Improve help and error messages when adding CVs to a CCV | ||
---|---|---|---|
Product: | Red Hat Satellite | Reporter: | Andrew Kofink <akofink> |
Component: | Content Views | Assignee: | Michael Johnson <micjohns> |
Status: | CLOSED ERRATA | QA Contact: | Lai <ltran> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 6.2.0 | CC: | ltran, micjohns, mpusater, pcreech |
Target Milestone: | 6.5.0 | Keywords: | EasyFix, Reopened, Triaged, UserExperience |
Target Release: | Unused | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2019-05-14 12:36:57 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Andrew Kofink
2018-02-02 13:24:30 UTC
Created from redmine issue http://projects.theforeman.org/issues/22498 Upstream bug assigned to None @Andrew, I spoke with Jonathon and he feels that the fix to the help descriptions is out of scope for this fix. Could you open another redmine issue for that since you made the initial request? Thanks! Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/22498 has been resolved. Set up a ccv and a cv. # hammer content-view add-version --id 12 --content-view-version-id 2 The component version has been added. Published the a new version of cv and reran the command # hammer content-view add-version --id 12 --content-view-version-id 13 Could not add version: Validation failed: Base Another component already includes content view with ID 2 @Michael - Why is the word "Base" included in the error message? Failed. Version Tested: Satellite-6.5 Snap 5.0 Michael, Here's the redmine issue for the help descriptions https://projects.theforeman.org/issues/25588. Sorry I didn't see this earlier! @Lai Rails is returning that by default. The previous change to the code overrode some of the default actions and now we are faced with this. I found a fix to the code, but it would require changing all of the error messages coming out of the content view components model, and I think that is going to cause more change/problems than it's worth. It will require changing the structure of the error messages and completely rewriting them. I would suggest leaving as-is because I believe 99% of people are going to read over the "Base" in that sentence. And I believe 100% of them will understand exactly what the sentence means. Updating the code on the other hand will increase the risk of more problems. @Michael @Andrew Since there's no changes to this issue and we're leaving it as-is, is this going to move back to ON_QA or is there more discussion happening around this? Removing the help text upstream issue so this can go back ON_QA Steps to test: Set up a ccv and a cv. # hammer content-view add-version --id 2 --content-view-version-id 2 The component version has been added. Published the a new version of cv and reran the command # hammer content-view add-version --id 2 --content-view-version-id 3 Could not add version: Validation failed: Base Another component already includes content view with ID 3 Version Tested: 6.5.0 Snap 14. Marking issue as verified. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2019:1222 |