-
-
Notifications
You must be signed in to change notification settings - Fork 909
feat: update create-react-app base to 5 and correct tooling issues #1342
feat: update create-react-app base to 5 and correct tooling issues #1342
Conversation
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.
Left a few comments. Just curious about a few things. Also generally curious about the reportWebVitals.js file and how that is being used.
@@ -26,7 +26,7 @@ function Repository({user, match}) { | |||
} = match; | |||
const [repository, setRepository] = useState(null); | |||
const [error, setError] = useState(null); | |||
const [note, setNote] = useState(location.note); | |||
const [note, setNote] = useState(null); |
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.
Curious about this as well. Does this solve an error that was overlooked?
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.
I’ve made this adjustment offline in the vite branch in response to some errors. but haven’t committed or pushed. I don’t know what location.note
Referred to but being that it was the prefetch state value something “empty” makes sense to have here.
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.
location.note
was never really populated - that key doesn't exist in the location object
@bdougie this was missing some commits :D I'll cancel the release flow and re-open if that's ok 🍕 |
…-5-webpack-5 * origin/main: feat: update create-react-app base to 5 and correct tooling issues (open-sauced#1342)
## [0.41.0](v0.40.2...v0.41.0) (2021-12-20) ### 🍕 Features * update create-react-app base to 5 and correct tooling issues ([#1342](#1342)) ([d30044c](d30044c)) ### 🐛 Bug Fixes * update create-react-app base to 5 and correct tooling issues ([#1344](#1344)) ([0fee440](0fee440)), closes [#1131](#1131) [#1343](#1343)
* origin/main: chore(minor): release 0.41.0 [skip ci] fix: update create-react-app base to 5 and correct tooling issues (open-sauced#1344) feat: update create-react-app base to 5 and correct tooling issues (open-sauced#1342) chore(deps-dev): bump postcss-preset-env from 7.0.1 to 7.0.2 (open-sauced#1338) chore(patch): release 0.40.2 [skip ci] fix: correct docker build (open-sauced#1335) build: update deps and semantic release (open-sauced#1333)
What type of PR is this? (check all applicable)
Description
This PR does a bunch of things:
Related Tickets & Documents
closes #1131
Mobile & Desktop Screenshots/Recordings
Added tests?
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?