Skip to content
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.

Sign up for GitHub

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

Jump to bottom

DO NOT MERGE: Observables -> React Query refactor #514

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

DO NOT MERGE: Observables -> React Query refactor #514

wants to merge 45 commits into from

Conversation

pingustar
Copy link
Collaborator

@pingustar pingustar commented Jun 22, 2022 •
edited by zavelevsky
Loading

fixes #532

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 22, 2022 •
edited
Loading

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 492a776
Status: ✅  Deploy successful!
Preview URL: https://d2260c22.falafel.pages.dev
Branch Preview URL: https://feature-react-query.falafel.pages.dev

View logs

pingustar and others added 21 commits June 23, 2022 11:18
@pingustar
react-query tmp commit
f711874
@pingustar
react-query tmp commit
5620a4d
@pingustar
Merge branch 'main' into feature/react-query
dd5d377
@pingustar
Merge remote-tracking branch 'origin/feature/react-query' into featur…
40522df 
…e/react-query
@pingustar
fix merge conflicts
5ab79b3
@pingustar
Merge branch 'main' into feature/react-query
b8cf770
@pingustar
core functionality proof of concept
2ae3e54
@pingustar
remove alert when BNT
d60264a
@pingustar
introduce new api for getting pool data
97b579b
@pingustar
introduce new api for pool table and top pools
f8812a9
@pingustar
Merge branch 'main' into feature/react-query
9b36912
@pingustar
cleanup part 1
bc6a8b7
@pingustar
refactor vote to for usePoolPick react-query hook
5321734
@pingustar
remove unneeded zod dependency
ca2cd9a
@pingustar
remove unneeded zod dependency
c8b1573
@pingustar
remove unneeded error boundry component
f2a73a5
@pingustar
Merge branch 'main' into feature/react-query
3274bd6
@pingustar
remove unneeded debug / test code
01ff24d
@pingustar
remove unneeded debug / test code
d00a886
@pingustar
rename to useChainBalances.ts
4e627b9
@pingustar
cleanup file/folder structure for queries
7867fe9
@pingustar pingustar marked this pull request as ready for review July 19, 2022 12:34
@pingustar pingustar changed the title WIP: testing react-query DO NOT MERGE: Observables -> React Query refactor Jul 19, 2022
pingustar and others added 20 commits July 20, 2022 11:35
@pingustar
Merge branch 'main' into feature/react-query
0de50c1
@pingustar
update react-query to v4
bcaa638
@pingustar
add index file and default exports for queries folder
e8da648
@pingustar
extract query client provider in separate file
a385971
@pingustar
add depositingEnabled to usePoolPick
009ba72
@pingustar
cleanup
8b837a9
@pingustar
add rate to usePoolPick
675fc8a
@pingustar
remove unused code
7ca6b80
@pingustar
add sentry error reporting
77d702b
@pingustar
polish and small bug fixes
d8a9c86
@pingustar
fix vote approval tmp
ddfed89
@pingustar
Merge branch 'main' into feature/react-query
a08bef5
@pingustar
fix subscribeToObservables based on pathname
cc424a0
@pingustar
Merge remote-tracking branch 'origin/main' into feature/react-query
a9798c5 
# Conflicts:
#	.env.sample
#	src/elements/earn/pools/TopPools.tsx
#	src/elements/earn/pools/poolsTable/PoolsTable.tsx
#	src/router/BancorRouter.tsx
#	yarn.lock
@pingustar
align with main and resolve conflicts
cc9c571
@pingustar
fix react query keys and key factory
da9ab92
@pingustar
use trading liquidity from api and fix react query error boundary
81061ea
@zavelevsky
adapted deposit modal
68f692d
@pingustar
refactor useApproveModal.tsx to use no token interface
67e8122
@pingustar
signed commit test
492a776
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RanCohenn RanCohenn Awaiting requested review from RanCohenn

@zavelevsky zavelevsky Awaiting requested review from zavelevsky

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move out of observables
2 participants
@pingustar @zavelevsky