Fixes #37798 - Fix AngularJS handling of AK CVEs and add warning banner for multi-env AKs #11148
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What are the changes introduced in this pull request?
content_view_id
andenvironment_id
params which didn't match theenvironment
andcontent_view
nested params. Example:(note the nonmatching IDs)
The API previously compensated for this, until #11135. Now, we compensate for it again.
2. Add a warning banner to the activation key details page:
Considerations taken when implementing this change?
This is somewhat of a stopgap for the UI until we can get true multi-environment display built. The plan is
What are the testing steps for this pull request?
Create a multi-CV activation key
Make an edit to the description or other AK fields
Ensure that the AK remains a multi-CV AK (you can check by looking for the banner, or looking at the API response
multi_content_view_environment
valueEnsure that you can change an AK's single cv/lce with no errors
Ensure that you can change a multi-env AK's cv/lce in the web UI. This will cause it to become a single-env AK; this is expected (you didn't read that banner, did you? ;)