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.

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

fix: unlimit thegraph query amount #3391

Merged
merged 2 commits into from
Jun 23, 2021

Conversation

zhouhanseng
Copy link
Contributor

This is an urgent and quick fix.

thegraph api only returns default first 100 entities when first attribute is absent, which cause that the user can't retrieve his 101th red packets from history.

I will open another issue to discuss tech solution of List lazy loading later.

guanbinrui
guanbinrui previously approved these changes Jun 17, 2021
nuanyang233
nuanyang233 previously approved these changes Jun 18, 2021
@guanbinrui guanbinrui dismissed stale reviews from nuanyang233 and themself via b4697a2 June 21, 2021 06:18
@guanbinrui guanbinrui force-pushed the hancheng/the-graph-query-amount branch from 95f66bf to b4697a2 Compare June 21, 2021 06:18
@guanbinrui guanbinrui changed the base branch from master to hotfix/1.33.2 June 21, 2021 06:19
@guanbinrui guanbinrui merged commit 4812412 into hotfix/1.33.2 Jun 23, 2021
@guanbinrui guanbinrui deleted the hancheng/the-graph-query-amount branch June 23, 2021 07:11
guanbinrui pushed a commit that referenced this pull request Jun 29, 2021
* fix: thegraph query amount limit

* chore: reply code review
guanbinrui added a commit that referenced this pull request Jun 29, 2021
* chore: bump version from 1.33.1 to 1.33.2

* refactor: remove useless code

* feat: transaction fee ui (#3321)

* fix: reset gas price (#3390)

* fix: reset gas price

reset gas price when leave tx confirm dialog or send the tx

* chore: little fix

* chore: only on eth mainnet

* fix: firefox does not connect correctly close jira-42 (#3413)

* fix: firefox does not connect correctly close jira-42

* Update packages/maskbook/src/extension/background-script/EthereumServices/send.ts

Co-authored-by: guanbinrui <[email protected]>

Co-authored-by: guanbinrui <[email protected]>

* fix: ito configured data will not be cleared after close ITO dialog (#3411)

* fix: ito configured data will not be cleared after close ITO dialog

* chore: solve conflict

Co-authored-by: Hancheng Zhou <[email protected]>

* fix: the text on welcome page can not display fully with small window width (#3410)

* fix: user can create persona by more than 24 characters (#3408)

* fix: clear poolSettings crash (#3442)

* fix: is switchable (close #3423)

* feat: ITO bsc polygon history (#3421)

* feat: ITO bsc polygon history

* fix: ui

* share dialog style
* token symbol

* fix: ui

* fix: variable

Co-authored-by: guanbinrui <[email protected]>
Co-authored-by: guanbinrui <[email protected]>

* fix: import path

* fix: should dependency urls and suffix when useAsync in useImageFailover (#3473)

* fix: unlimit thegraph query amount (#3391)

* fix: thegraph query amount limit

* chore: reply code review

* fix: user can create persona by more than 24 characters (#3477)

* fix: the create button is unable to click, press enter key to create

* Apply suggestions from code review

Co-authored-by: Septs <[email protected]>

* refactor: reduce block number queries (#3511)

* chore: update weth address

* refactor: move chain state to wallet plugin

* chore: add useStartWatchChainState

* fix: fetch balance if wallet is just created

* refactor: batch requests

* refactor: remove unused deps

Co-authored-by: Jack Works <[email protected]>

* refactor: reply reviews

* fix: chain name

Co-authored-by: Jack Works <[email protected]>

* fix: lint error

Co-authored-by: maskbook-maintainer-bot[bot] <66122279+maskbook-maintainer-bot[bot]@users.noreply.github.com>
Co-authored-by: guanbinrui <[email protected]>
Co-authored-by: Jack Works <[email protected]>
Co-authored-by: guanbinrui <[email protected]>
Co-authored-by: lelenei <[email protected]>
Co-authored-by: Hancheng Zhou <[email protected]>
Co-authored-by: Lantt <[email protected]>
Co-authored-by: Septs <[email protected]>
guanbinrui added a commit that referenced this pull request Jul 1, 2021
* fix: thegraph query amount limit

* chore: reply code review
Jack-Works added a commit that referenced this pull request Jul 1, 2021
* chore: bump version from 1.33.1 to 1.33.2

* refactor: remove useless code

* feat: transaction fee ui (#3321)

* fix: reset gas price (#3390)

* fix: reset gas price

reset gas price when leave tx confirm dialog or send the tx

* chore: little fix

* chore: only on eth mainnet

* fix: the text on welcome page can not display fully with small window width (#3410)

* fix: user can create persona by more than 24 characters (#3408)

* fix: is switchable (close #3423)

* feat: ITO bsc polygon history (#3421)

* feat: ITO bsc polygon history

* fix: ui

* share dialog style
* token symbol

* fix: ui

* fix: variable

Co-authored-by: guanbinrui <[email protected]>
Co-authored-by: guanbinrui <[email protected]>

* fix: should dependency urls and suffix when useAsync in useImageFailover (#3473)

* fix: unlimit thegraph query amount (#3391)

* fix: thegraph query amount limit

* chore: reply code review

* fix: user can create persona by more than 24 characters (#3477)

* fix: the create button is unable to click, press enter key to create

* Apply suggestions from code review

Co-authored-by: Septs <[email protected]>

* refactor: reduce block number queries (#3511)

* chore: update weth address

* refactor: move chain state to wallet plugin

* chore: add useStartWatchChainState

* fix: fetch balance if wallet is just created

* refactor: batch requests

* refactor: remove unused deps

Co-authored-by: Jack Works <[email protected]>

* refactor: reply reviews

* fix: chain name

Co-authored-by: Jack Works <[email protected]>

* fix: build error

* fix: ITO build errors

* fix: lint error

* chore: revert ito changes

Co-authored-by: maskbook-maintainer-bot[bot] <66122279+maskbook-maintainer-bot[bot]@users.noreply.github.com>
Co-authored-by: lelenei <[email protected]>
Co-authored-by: Lantt <[email protected]>
Co-authored-by: Septs <[email protected]>
Co-authored-by: Jack Works <[email protected]>
Jack-Works added a commit that referenced this pull request Jul 15, 2021
* chore: bump version from 1.33.1 to 1.33.2

* refactor: remove useless code

* feat: transaction fee ui (#3321)

* fix: reset gas price (#3390)

* fix: reset gas price

reset gas price when leave tx confirm dialog or send the tx

* chore: little fix

* chore: only on eth mainnet

* fix: the text on welcome page can not display fully with small window width (#3410)

* fix: user can create persona by more than 24 characters (#3408)

* fix: is switchable (close #3423)

* feat: ITO bsc polygon history (#3421)

* feat: ITO bsc polygon history

* fix: ui

* share dialog style
* token symbol

* fix: ui

* fix: variable

Co-authored-by: guanbinrui <[email protected]>
Co-authored-by: guanbinrui <[email protected]>

* fix: should dependency urls and suffix when useAsync in useImageFailover (#3473)

* fix: unlimit thegraph query amount (#3391)

* fix: thegraph query amount limit

* chore: reply code review

* fix: user can create persona by more than 24 characters (#3477)

* fix: the create button is unable to click, press enter key to create

* Apply suggestions from code review

Co-authored-by: Septs <[email protected]>

* refactor: reduce block number queries (#3511)

* chore: update weth address

* refactor: move chain state to wallet plugin

* chore: add useStartWatchChainState

* fix: fetch balance if wallet is just created

* refactor: batch requests

* refactor: remove unused deps

Co-authored-by: Jack Works <[email protected]>

* refactor: reply reviews

* fix: chain name

Co-authored-by: Jack Works <[email protected]>

* fix: build error

* fix: ITO build errors

* fix: lint error

* chore: revert ito changes

Co-authored-by: maskbook-maintainer-bot[bot] <66122279+maskbook-maintainer-bot[bot]@users.noreply.github.com>
Co-authored-by: lelenei <[email protected]>
Co-authored-by: Lantt <[email protected]>
Co-authored-by: Septs <[email protected]>
Co-authored-by: Jack Works <[email protected]>
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.

4 participants