Repository navigation
Conversation
Mentors currently cannot update their scheduling link from the profile editor even though the API already stores it. Expose the field in the shared edit form so mentors and admins can update the same value.
|
khairahscorner
left a comment
khairahscorner
left a comment
There was a problem hiding this comment.
LGTM, would be nice if we can get a screenshot of the updated view.
@Sowmiya07 can you review as well please?
|
@CreedTech - Can you please share more details why are you not able to test this change locally ? You should be able to run the admin app locally with the changes. As it is UI change it would be good to get some screenshots. |
|
I got the admin app running locally. I saved a scheduling link on the mentor’s profile, reloaded it, and confirmed the link also shows in the mentor’s My Profile view. Screenshots attached. |
|
Hi @Sowmiya07, could you take a look when you have time? The local admin-app screenshots and save/reload check are in #819 (comment). The frontend CI job is passing on the current head. This supports adding or replacing a scheduling link; clearing an existing link still needs API support. |
|
@goelsonali / @dricazenck can you review please as Sowmiya is off for some time? Thank you. |
Description
Add a scheduling link field to the shared mentor profile editor. Existing links load into the field, and a replacement is included when the profile is saved. The same editor is used by mentors and admins.
The current API retains an existing link when given a blank value, so this change supports replacing a link but not removing one.
Related Issue
Closes #818
Change Type
Screenshots
Tested in the local admin app: the scheduling link saved and remained after reloading the profile. It also appears in the mentor’s My Profile view. Screenshots are in this comment.
Pull request checklist
Focused form tests, type check, ESLint, Prettier, and the production build passed.