-
Notifications
You must be signed in to change notification settings - Fork 448
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
Add disabled property for form text in ui-library #8850
Comments
@withanage I've assigned this to the metadata and distribution team, because I think the motivation for this feature is the ORCID plugin. Is that right? |
@NateWr thanks! sure , it is ok for the Metadata and Distribution. The function is but generally a development what we need for the mail functions in CRAFT-OA project context. |
Thanks @withanage. I haven't heard about the mail functions. Can you describe the use case in more detail? |
Sure , I have added some context and the use-case ContextThis is part of the GDPR support TIB plans to co-develop with TSV (Anttil-Jussi) . For the GDPR support , the infrastructure support will be done by PKP team and the functionality and use-cases are developed by TIB. The requirement comes from pkp-lib#3022 I have not assigned it yet, cause we were thinking of a creating a meta-issue for GDPR and linking the small issues together. Use case
|
@withanage I think you need to discuss further what this is actually going to look like. There is similar work done in #7391 and @Devika008 has contributed a suggested design pattern for enabled/disabled sections of forms here: #7391 (comment). Can you follow up with a more detailed proposal for the proposed feature? I don't want to merge in a configuration option for |
@NateWr thanks for pointing to the #7391 and @Devika008's ui-plan. @Devika008 we will come back to you , as soon as we have the functional descriptions for the GDPR support. @NateWr as motivation for adding the functionality as a core-feature, I have taken the orcidProfile plugin use-case and updated the #8850 header with use-case description. |
Use case description
optInEdit
option.optInEdit
option adds an additional button, which in turn enables editing thedisabled
form text field, which is not acceptable by ORCiD requirements.Solution description
What application are you using?
OJS 3.5 +
Additional information
The text was updated successfully, but these errors were encountered: