-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade core dependencies #1089
Conversation
zubhub_frontend/zubhub/src/views/edit_profile/editProfileScripts.js
Outdated
Show resolved
Hide resolved
zubhub_frontend/zubhub/src/views/edit_profile/editProfileScripts.js
Outdated
Show resolved
Hide resolved
zubhub_frontend/zubhub/src/views/edit_profile/editProfileScripts.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did not really go through deeply but just some question with respect to the validation methods and messages
zubhub_frontend/zubhub/src/assets/js/styles/views/page_wrapper/pageWrapperStyles.js
Show resolved
Hide resolved
zubhub_frontend/zubhub/src/views/edit_profile/editProfileScripts.js
Outdated
Show resolved
Hide resolved
zubhub_frontend/zubhub/src/views/edit_profile/editProfileScripts.js
Outdated
Show resolved
Hide resolved
0cf585c
to
817092f
Compare
b4909d3
to
70daa3e
Compare
70daa3e
to
c67c98b
Compare
@NdibeRaymond is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let some more reviews @coderatomy @brrkrmn . The most important ones are the ones around in PageWrapper.jsx
and App.jsx
zubhub_frontend/zubhub/src/views/create_project/CreateProject.jsx
Outdated
Show resolved
Hide resolved
@brrkrmn and @coderatomy there are some reviews that I mentioned people specifically or insinuated that someone introduced the change being reviewed. If I mentioned you but it wasn't you who introduced the change, just ignore that I mentioned you and help fix the issue if you can. |
@coderatomy is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
@coderatomy is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
@coderatomy is deleting test VM ... |
Test VM is ready ✅✅✅ |
@coderatomy is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
83712f3
to
d257062
Compare
@NdibeRaymond is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
@NdibeRaymond is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
14d42fc
to
d257062
Compare
@NdibeRaymond is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
@NdibeRaymond is deleting test VM ... |
Test VM deleted ✅✅✅ |
@NdibeRaymond is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
d257062
to
9a64ac3
Compare
@coderatomy is creating a test VM for this PR 🚀🚀🚀 |
Test VM is ready ✅✅✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merging
@NdibeRaymond is deleting test VM ... |
Test VM deleted ✅✅✅ |
Revision for #1048
The first commit includes changes related to import paths, from
history
tonavigate
and fromprops.match
toprops
changes while the second commit includes files with other functional changes.