EPerson creation page changed#2130
Conversation
While creating a new EPerson - * Reset password button is not displayed. * Submit button label changed from 'Save' to 'Create'.
This reverts commit feb2d09.
This reverts commit 0090088.
While creating a new EPerson - * Reset password button is not displayed. * Submit button label changed from 'Save' to 'Create'.
|
@amanbudgujar : Just wanted to let you know this PR is failing a "lint" check cause of trailing spaces. You can find the details of the error by running |
|
@tdonohue |
|
@tdonohue |
|
@amanbudgujar : This PR is on our 7.6 board and is waiting on reviewers to be assigned. As all reviewers are volunteers, we unfortunately cannot promise a timeline for when the review will occur. But, we can promise this fix will be reviewed in time for 7.6. Since it's a small PR though, hopefully we can find someone sooner rather than later (or maybe I'll find time to review it myself). If you'd like to help our review process along, we accept PR feedback from anyone. So, you are welcome to help by volunteering to review or simply test (if you don't feel comfortable with code reviews yet) the PRs of others. That can help us keep our backlog smaller and allow us to get to your PRs more quickly as well. Thanks and hopefully I or someone else will get back to this PR soon. |
tdonohue
left a comment
There was a problem hiding this comment.
👍 Thanks @amanbudgujar ! I finally got around to reviewing/testing this today. It works well, and I like that the creation page now has a "Create" button instead of a "Save" button.
References
Description
While creating a new EPerson -
Instructions for Reviewers
Steps to test
List of changes in this PR:
en.json5file.eperson-form.component.htmlfileeperson-form.component.tsfile.Checklist
yarn lintyarn check-circ-deps)package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.