[Remove Vuetify from Studio] 'Forgot password' page - #6131
[Remove Vuetify from Studio] 'Forgot password' page#6131Vaibhavkumar-tech wants to merge 2 commits into
Conversation
|
👋 Hi @Vaibhavkumar-tech, thanks for contributing! For the review process to begin, please verify that the following is satisfied:
Also check that issue requirements are satisfied & you ran Pull requests that don't follow the guidelines will be closed. Reviewer assignment can take up to 2 weeks. |
|
@akolson I've opened the PR with the changes for the Forgot Password page. I've kept the changes limited to the page as requested and followed the validation approach discussed above. When you have a chance, could you please review it and let me know if any changes or improvements are needed? Thank you! |
There was a problem hiding this comment.
Hi @Vaibhavkumar-tech! Great changes on this. Code changes makes sense to me! However, it appears that there is a noticeable/major change(and not minor as suggested) in the sizing of the field and submit button comparing with what is currently present in production. Applying appropriate css styling should resolve this.


Summary
Migrated the Forgot Password page from the legacy Vuetify components to the Studio/KDS components.
MessageLayoutwithStudioMessageLayout.EmailFieldwithStudioEmailField.BannerwithStudioBanner.VFormand its validation usage.References
Reviewer guidance
Manual verification
UI
Minor visual differences from the previous Vuetify implementation are expected because the page now uses Studio/KDS components.
AI usage
I used ChatGPT to help understand the existing Vuetify implementation, the Studio components, and the project's form-validation pattern. I reviewed the generated suggestions critically, compared them with the existing code and maintainer guidance, and kept the implementation limited to the Forgot Password page as requested.