-
Notifications
You must be signed in to change notification settings - Fork 409
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
#9583 unadvertised resources #9962
#9583 unadvertised resources #9962
Conversation
Added unadvertised field in Metadata.jsx, shown in modal form. Checkbox, controlled. Adjust handleResourceData.jsx enhancer, include unadvertised field from resource originalData unto props passed to Metadata.jsx. Added translations for unadvertised field.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed and tested 👍
@Igi-ID for your future work, take into account that only mandatory localization files are strictly requied (it,en,us,fr,de)
waiting for build to pass (there is a test that fails often and it should be taken into account @tdipisa , I opened an issue for this problem. As well as deployed, please @ElenaGallo test on dev |
@ElenaGallo yes, see here for the AC clarification #9583 (comment) in respect of the usual MS overall model. |
Just one thing I noticed @ElenaGallo @Igi-ID If the unadvertised checkbox has been previously checked, when you open the map as a user without edit rights (e.g. logout with user with edit rights and then login again with a different user without edit rights), that checkbox is still checked in the Edit properties opening when clicking on the Save as button. The state should be reset, I think: |
Fix provided here #9984 |
Description
This PR introduces the "Unadvertised" field on creation / edit of resources. Resources which are tagged as unadvertised are viewable and editable only by the owner or admin role users.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
#9583 Unadvertised Resources
What is the new behavior?
The "Unadvertised" field is viewable and togglable inside the resource modal on creation/editing. Its value is processed and persisted and resources are filtered accordingly.
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information