Skip to content

Feature/redux#1

Open
wialy wants to merge 4 commits intomainfrom
feature/redux
Open

Feature/redux#1
wialy wants to merge 4 commits intomainfrom
feature/redux

Conversation

@wialy
Copy link
Copy Markdown
Collaborator

@wialy wialy commented Feb 8, 2021

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
Copy link
Copy Markdown
Collaborator

ArekRado commented Feb 9, 2021

@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