VRMS icon indicating copy to clipboard operation
VRMS copied to clipboard

editMeetingTimes - Replaced HTML with MUI applicable components

Open nora-zajzon opened this issue 1 year ago • 4 comments

Fixes #1665

What changes did you make and why did you make them ?

  • Replaced all HTML components with MUI components in /src/components/manageProjects/editMeetingTimes.js

nora-zajzon avatar Jul 17 '24 17:07 nora-zajzon

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b nora-zajzon-editMeetingTimes development
git pull https://github.com/nora-zajzon/VRMS.git editMeetingTimes

github-actions[bot] avatar Jul 17 '24 17:07 github-actions[bot]

@JackHaeg can I ping you to track that this component needs to be transferred to a modal provided by MUI? I think we already have an issue open for it

trillium avatar Aug 05 '24 22:08 trillium

@trillium to create tracking issue

JackHaeg avatar Aug 07 '24 18:08 JackHaeg

  • New tracking issue created by @trillium: #1724

JackHaeg avatar Aug 08 '24 20:08 JackHaeg

@trillium Please take a look at this to check for duplicate work. Since this is fixing multiple issues, and some of the issues have been tackled by other members, we might be grabbing specific commits from this PR and merging those.

JackHaeg avatar Feb 20 '25 21:02 JackHaeg

@trillium Please take a look at this to check for duplicate work. Since this PR is fixing multiple issues, and some of the issues have been tackled by other members, we might be grabbing specific commits from this PR and merging those.

JackHaeg avatar Mar 04 '25 02:03 JackHaeg

Per @trillium's direction, we are closing this PR due to merge conflicts and duplicate changes that will be difficult to overcome due to more recent edits on VRMS. This PR will be used as a reference when creating an updated fresh PR should be submitted

JackHaeg avatar May 08 '25 22:05 JackHaeg