Skip to content

Feature/redux - #1

Open
wialy wants to merge 4 commits into
mainfrom
feature/redux
Open

wialy wants to merge 4 commits into
mainfrom
feature/redux

Conversation

@wialy

@wialy wialy commented Feb 8, 2021

Copy link
Copy Markdown
Collaborator

No description provided.

@wialy
wialy requested review from ArekRado and Stosiu February 8, 2021 19:24
@wialy wialy self-assigned this Feb 8, 2021
@ArekRado

ArekRado commented Feb 9, 2021

Copy link
Copy Markdown
Collaborator

@wialy I strongly recommend to not store api responses in redux. Libraries like a react-query uses their own cache reducing amount of boilerplate. I think whole PR will be much smaller if we will use react-query instead. I use it on my projects and it's awesome.

Can we add react-query to our frontend-template? What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants