-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat: use indexer directly [JS-842] #1069
Merged
+5,237
−1,327
Merged
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
771f61e
feat: refactor configs [JS-831]
shamsartem 38455bc
fix circular dependency
shamsartem 7a61077
Apply automatic changes
shamsartem 2fdc977
fix
shamsartem 7991aeb
Apply automatic changes
shamsartem 3dc1d8d
simplify
shamsartem 0ebac89
import only what's needed from lodash
shamsartem 75601cf
fix?
shamsartem 5504ab5
just in case
shamsartem e804ef1
Merge branch 'main' into refactor
shamsartem 7d9c540
Merge branch 'main' into refactor
shamsartem da98076
make all aliases hidden from top-level of help
shamsartem 2d4acd2
Apply automatic changes
shamsartem 41bcde0
stop managing ocker compose config completely
shamsartem d49a1f9
feat: use indexer directly
shamsartem 616479f
fix
shamsartem eb52c51
don't require build for linting
shamsartem 735d792
remove irrelevant dependencies
shamsartem be52e49
add docker for on-each-commit
shamsartem 8856f00
add env
shamsartem fba4662
change runs-on
shamsartem 5fdd23e
add permissions?
shamsartem 22f6ada
fix and remove stuff from docs.yml
shamsartem c1530c8
fix
shamsartem ebfeec1
add generated to prettierignore
shamsartem a19187e
add config promise to the Map right away to prevent possible problems…
shamsartem 53a68b0
fix
shamsartem 278dcad
remove rest of the clients
shamsartem e3401d9
Merge branch 'main' into indexer
shamsartem 65afa0f
get cc statuses from RPC
shamsartem 9b0c2f7
lowercase dealId
shamsartem 43dc831
add debug log
shamsartem 6f70f4d
try using batch read like before instead of multicall read
shamsartem a7ee43f
try adding timeout
shamsartem 7f53cee
use on-chain info for cc, refactor and fix
shamsartem cafff89
Apply automatic changes
shamsartem a194ec7
take ccId from subgraph if cc from RPC is zeroHash (was removed)
shamsartem 9be7d30
don't convert already converted peerId
shamsartem b7c2a47
Merge branch 'main' into indexer
shamsartem 20cb1b5
rename left-ower Fluence Labs
shamsartem c354632
improve startDate and expirationDate display
shamsartem File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev
Previous commit
Loading status checks…
improve startDate and expirationDate display
commit c35463221fe5b13e9ad625931c787e4198ab0430
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's split this file into smaller ones
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you don't mind I will think about doing that in a separate PR. I think splitting the code in multiple files doesn't solve any concrete issues, more of a matter of a taste thing, though I could be wrong and I agree this file is pretty big already. On the other hand it still seems ok to work with and as a bonus doesn't require you to jump between many files when working with commitments, which is pretty convenient