Jack Haeger
Jack Haeger
Hi @angela-lee1 No worries at all - seriously!! Thanks so much for confirming and for making these updates to the PR :)
@angela-lee1 Just checking in since I saw an the additional commit submitted last week. Is this PR ready for review? Based on your last comment, it sounds like you might...
Per @angela-lee1 this PR is ready for review.
@angela-lee1 To resolve merge conflicts
Hi @angela-lee1 - Just checking in! Can you please provide an update on when you believe these merge conflicts will be addressed?
Hi @angela-lee1 just checking in on this PR :) Whenever you have a moment, can you please resolve these merge conflicts so that we can merge this PR?
@ntrehan & @trillium - looks like the updated MUI buttons used in the above screenshots are no longer viewable to the user (no outline viewable on the button, only button...
@trillium Do you think we should change this to an outlined button, like I mentioned in my previous comment before we go ahead and merge?
@ntrehan, confirmed with @trillium that we should change the button style to contain borders - please make the updates according to the above comment: https://github.com/hackforla/VRMS/pull/1725#issuecomment-2292034373
@trillium agrees with the changes above. Any team member can review this issue.