Closed JoaoFGuiomar closed 9 months ago
@JoaoFGuiomar is attempting to deploy a commit to the ergo-pad Team on Vercel.
A member of the Team first needs to authorize it.
This is great. It will take us a bit to review all this but thank you, some of those are small annoyances that just needed a good cleanup. The "i" and "c" being used in map functions was a bit of a pet peeve so I like to see those getting replaced with something a bit more verbose ("i" should be used for index imo). Tons of bigger improvements too.
@esot321c Rebased from beta
and solved conflicts. Sorry about the long PR
Awesome work!!! But we could not merge this due to the large diff. But I have manually merged the changes in beta branch. Please reach out to @esot321c for the bounty.
Description
This PR was supposed to only add a
ErrorPage
.Eventually, it got out of hand 😅 :
Emotion
theme while working withstyled
Providers
component that gathers all context providers in one component.AppApi
arguments so it receives theuseState
tuple instead of each value individually. I think it just makes it a bit more readable, no functionality should change.themeState
,loadingState
,daoState
,daoUserState
,alertState
instead ofloading
,setLoading
,theme
,setTheme
,aler
,setAlert
and so on.Husky
forpre-commit
lintspackage.json
andnvmrc
Related Issue
Related PR
Screenshots