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

[RFR] create account token snapshot log for each account token snapshot updates #830

Conversation

msoni89
Copy link
Contributor

@msoni89 msoni89 commented May 18, 2022

No description provided.

@msoni89 msoni89 changed the title [WIP] create account token snapshot log for each account token snapshot updates [RFR] create account token snapshot log for each account token snapshot updates May 18, 2022
msoni89 added 4 commits May 18, 2022 12:17
…h_account_token_snapshot_update

# Conflicts:
#	packages/subgraph/src/mappingHelpers.ts
#	packages/subgraph/test/validation/aggregateValidators.ts
Copy link
Contributor

@0xdavinchee 0xdavinchee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks dope overall, just a few changes

packages/subgraph/schema.graphql Outdated Show resolved Hide resolved
packages/subgraph/schema.graphql Outdated Show resolved Hide resolved
packages/subgraph/src/mappings/idav1.ts Show resolved Hide resolved
packages/subgraph/src/utils.ts Outdated Show resolved Hide resolved
packages/subgraph/test/interfaces.ts Outdated Show resolved Hide resolved
packages/subgraph/test/validation/aggregateValidators.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@0xdavinchee 0xdavinchee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good LGTM

@0xdavinchee 0xdavinchee merged commit 5073b90 into dev May 18, 2022
@0xdavinchee 0xdavinchee deleted the 775_create_account_token_snapshot_log_for_each_account_token_snapshot_update branch May 18, 2022 13:11
@github-actions
Copy link

XKCD Comic Relif

Link: https://xkcd.com/830
https://xkcd.com/830

* Log entities util functions
*************************************************************************/

export function createLogID(
Copy link
Contributor

@kasparkallas kasparkallas May 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would have been neat if this incremented based on account updates...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this may be useful so the IDs can be sortable?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The database would automatically order the ID-s a bit better, yes, but it depends on how the database is set up, regarding how it orders "-5" vs "-12". If it prioritizes shorter length then the ID-s would be perfectly sorted at the time of insertion, yes. This can have some query performance implications.

The other small advantage is that it's easy to see how many updates the account has had. This might make planning for queries easier in some cases... Also could be useful information in general.

I'm not saying this is a big deal at all... Just discussing and saying an indexer based ID would have felt more natural to me. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean by indexer-based ID?

We could append an order id as part of the ID (generated based on #805).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean like id-1, id-2, id-3, id-4, id-5... etc after every change.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want it to store in memory or a database?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking just having a version or updateCounter on AccountTokenSnapshot entity which gets put into the ID of the log entry.

Do you guys think it could be useful? It feels natural to me...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, why not.

Sure let's add one more property to track updates count, please create a separate ticket. Also regarding triggered event: Event! Please mention what properties you want to keep in the HOL entity.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One problem is, that the code is pretty much scattered updating from multiple places and due to which maybe be counter or version possibly need to pass a parameter.

"""
AccountTokenSnapshotLog: Historical entries of AccountTokenSnapshot updates.
"""
type AccountTokenSnapshotLog @entity {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be marked immutable somewhere? @0xdavinchee

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not super necessary tbh

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it's supposed to be an immutable entity ;)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's add this to an issue and give the HOL entities an interface to implement and do this once the graph fixes the issue w/ mixing immutable/non-immutable

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently it got fixed: graphprotocol/graph-node#3553 ;) Maybe hasn't shown up in hosted service yet.

transactionHash: Bytes!
logIndex: BigInt!
order: BigInt!
triggeredByEventName: String!
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@0xdavinchee Could we use triggeredByEvent: Event! here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, but it may be useful to have both properties, one for filtering and the other for the information

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess... although in this case, the event ID actually contains all the same information, except the order, but of course it's not as explicit.

By the way, it seems that the Graph team is quite close to implementing filtering by child entity: graphprotocol/graph-node#3184

Once that gets merged, we can have a more normalized schema, i.e. less duplicated fields just for filtering.

0xdavinchee added a commit that referenced this pull request May 23, 2022
* Add few more CFA and IDA cases (#781)

* SDK-Core and Subgraph Release Cleanup (#784)

* subgraph clean up
* make FlowOperatorUpdatedEvent immutable
* fix comment
* supertoken initialization wip
* added loadNativeAssetSuperToken, loadPureSuperToken and loadWrapperSuperToken functions in Framework
* loadSuperToken returns base SuperToken class
* added tests for new initialization
* move createFlowOperatorUpdatedEventEntity after flowOperator entity is created

* SDK-Core Version 0.4.0 (#782)

* changelog breaking added

* removed most of the readme file text

* update dependency versions

* update package.json dependency versions
* piggyback incorrect change-log versions
* update sdk-core changelog accordingly
* update sdk-core version in subgraph

* bump sdk-core version in sdk-redux

* update js-sdk version and yarn.lock

* Update CHANGELOG.md

* Update CHANGELOG.md

* update package versions for examples

* fix broken example build

* Update yarn.lock

* same as simple-acl-close

* forgot about yarn.lock file...

* fix transactiontable.tsx

Co-authored-by: Miao ZhiCheng <[email protected]>

* Refactor SuperfluidFrameworkDeployer & ERC1820RegistryCompiled (#789)

* refactor SuperfluidFrameworkDeployer & ERC1820 support
* include cfa/ida libs in the framework struct

* Simple ACL Gelato Example (#779)

* rename files
* fix forge build after copy pasta
* clean up Makefile
* local .gitmodules and remove .gitmodules in root
* comment out ida deployment as this is silently breaking
* add hardhat.config.ts
* create OpsMock contract
* setOps in resolver
* add tests for SimpleACLCloseResolver
* cleanup setup()
* setup cleanup
* test case cleanup
* remove superfluidtester file
* cleanup test file: remove comments, organize by section
* add more tests
* add more TODO tests
* add more functionality to simpleaclcloseresolver
* added a bunch of fuzzing tests because why not?
* added some general test cases
* add README.md
* add forge deploy in make file
* add check if flow exists for `canExec` boolean
* modify hardhat.config.ts to work with foundry test files
* fix tests accordingly (block.timestamp starts from 0)
* add hardhat-ethers, sdk-core and hardhat-etherscan
* hardhat deploy script added
* GAS_LIMIT added to .env.tempalte
* typechain-types added to .gitignore
* typechain added to hardhat config
* tsconfig added
* added scripts to package.json
* added verifyContract hardhat task
* modified README.md
* deploy script fleshed out
* remove gasLimit in resolver
* quick tests setup
* basic test setup
* build-and-test.sh script added
* package.json formatting
* deploy framework and tokens
* SimpleACLResolver tests added
* deploy script cleanup
* SimpleACLResolver.t.sol naming cleanup
* ethers required otherwise typings is weird
* add gas reporter/solidity coverage for hardhat

* Bump ethereum-contracts to 1.2.2 (#791)

* Employment Loan Example: Patch lend function (#786)

* Subgraph 1.3.2 patch (#794)

* I think this is leading to issues w/ subgraph breaking

* actual fix

* this was creating 0 address account token snapshots as it always saves

* update subgraph version

* test case added

ensure no Account or AccountTokenSnapshot entities with the zero address

* missing :

* query mistake

* cleanup + clarification

* allow zero address account/ATS

* incorrect test description

* Subgraph Fix Pt2 (#798)

* I think this is leading to issues w/ subgraph breaking

* actual fix

* this was creating 0 address account token snapshots as it always saves

* update subgraph version

* test case added

ensure no Account or AccountTokenSnapshot entities with the zero address

* missing :

* query mistake

* cleanup + clarification

* allow zero address account/ATS

* incorrect test description

* remove immutable: true test

* test theory

* Revert "test theory"

This reverts commit 5ae2ebd.

* add tests

* test pulling all the subgraph entities which have an `ILightEntity` and ensure that none of them break

* print out the retrieved data

* Update ACL example (#799)

* update ACL example

* update ethereum-contracts to v1.2.2
* utilize ERC1820RegistryCompiled
* fix SFDeployer to take new syntax
* fix Makefile commands

* use cfaLib from sfFramework

* use CFAv1Lib from SFDeployer

* Js-SDK maintenance branch release 0.6.0 (#802)

* Fix a query syntax in subgraph testing (#801)

* fix coverage test path issue

* fix ci: set working-directory correctly

* fix ci

* fix ci

* [skip ci] Add a NOTE to ci

* [778] SlotsBitmapLibrary Property Test Fuzzing (#790)

* CfAv1Library ACL support (#803)

* [RFR] Added logIndex property to events (#800)

* Fix codecov source files (#806)

* [RFR] 654 added a property to order events chronologically (#805)

* Fix ethereum-coverage code coverage  (#808)

* New package: hot-fuzz, a hot fuzz for testing Superfluid protocol and Super Apps (#708)

* Fix the flowlottery hot fuzz temp link (#812)

* add test to hardhat budget nft suite for transferring NFT ownership (#795)

Co-authored-by: Miao ZhiCheng <[email protected]>
Co-authored-by: 0xdavinchee <[email protected]>

* Some small changes (#813)

* docs: add bertux as a contributor for review (#820)

* Hotfuzz improvements (#814)

* update hot-fuzz version due to breaking change

* avalanche-c (#823)

* Avalanche Support Added (#819)

* avalanche support added

* avalanche support added for ethereum-contracts (hardhat), sdk-core in constants.ts and subgraph (config/, networks.json, addresses.template.ts)
* subgraph `README.md` cleanup
* deploy-all-networks variable renaming

* constants for chainId

* js-sdk getConfig update

* add avalanche to networkNameToChainIdMap

* goerli => matic rename

* canonical naming for avalanche-c

* added Avalanche to supertoken deployer Dapp

* readme update

* use canonical naming from truffle-config.js

* use truffle-config canonical names in constants for sdk-core

* fix empty pk case

* matic => polygon-mainnet for sdk-core tests

* new js-sdk/sdk-core versions bumped!

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

* [RFR] Added criticalAtTimestamp entity to subgraph v1 (#822)

* new field added maybeCriticalAtTimestamp account token snapshot
* Added maybeCriticalAtTimestamp account token snapshot
* added to SDK-Core subgraph files
* changed variable name to MAX_SAFE_MILLISECONDS

* [RFR] create account token snapshot log for each account token snapshot updates (#830)

* added logs entitt
* added code for account balance log
* fixed --  added test cases
* fixed ida broken test cases
* comment added
* merged critical at timestamp changes into branch
* added isLiquidationEstimateOptimistic flag
* updated:: as per review comment

* Create folder structure for examples (#804)

* Fixed:: TypeScript error in /Users/mikeghen/Documents/Projects/ricochet-frontend/node_modules/@superfluid-finance/js-sdk/src/InstantDistributionAgreementV1Helper.d.ts(197,19): (#829)

* allow same app registration on testnets and mainnets

* Sdkcore updates new fields and entities changes (#834)

* added account token balance logs and missing field from other story

* Revert "Sdkcore updates new fields and entities changes (#834)" (#836)

This reverts commit cd892eb.

* SDK-Core Release Subgraph Compatibility Tests (#835)

* add subgraph tests in sdk-core tests
* general ci cleanup: quote consistency, remove unused env variables
generate-graphql-schema for correct release in feature, canary and release
use hardhat node instead of truffle and update hardhat.config.ts to point to hardhat node chainId
separate startGanacheAndDeployContracts into startHardhatNode and setupTestEnvironment
* add multiple versions test to ci-pre-release
refactor 5_subgraph_test.ts by taking test functions out
new previous-versions-testing folder which contains queryTests.ts (extracted functions from above), runQueryTests.ts (for running previous-version-tests) and package.json which has sdk-core as a devDep
* generate-graphql-schema before build and release
* only test live subgraph release for sdk-core-release and only test previous sdk-version releases for subgraph release
* use query instead of framework for subgraph query tests + add console.log to see where things are breaking-local subgraph taking too long
* try reusable workflow, use query instead of full framework object
* test and deploy subgraph reusable workflow takes inputs, ci.canary, ci.feature, ci.pre-release cleanup w/ use of reusable workflow, use goerli TEMPORARILY because matic indexing is really slow
* rename call.setup-and-deploy to setup-deploy-and-test, cleanup + uses fix, testing https://github.blog/changelog/2022-01-25-github-actions-reusable-workflows-can-be-referenced-locally/, add test-subgraph-on-previous-sdk-versions to canary build and subgraph deploy, use local-subgraph-rule in test-previous-versions for deploy-subgraph case, LOCAL_SUBGRAPH_URL taken for runQueryTests
* check indexing completeness script added and reusable workflow added for it, deploy subgraph needs build and test w/ previous sdk-core releases to pass
* use matic instead of goerli
* workflow name headline consistency + double quote consistency cleanup + caller files added to dependent paths

* Fix workflow (#839)

* Yellow Paper Latex Pipeline

* Last CI fix  (#840)

* Local subgraph + SDK-Core tests fixes + cleanup

* Sdkcore updates new fields and entities changes (#837)

* Revert "Revert "Sdkcore updates new fields and entities changes (#834)" (#836)"

This reverts commit f8cb3ec.

* require test-subgraph-on-previous-sdk-versions

add test-subgraph-on-previous-sdk-versions to needs before publish-npm-packages job is run

* remove test-subgraph-on-previous-sdk-versions job from ci.canary

Co-authored-by: 0xdavinchee <[email protected]>

* CI Pre Release Draft Fix + CI Canary Cleanup (#843)

* cleanup + fix logic of build files

* generate dev schema + SUBGRAPH_RELEASE_TAG: dev

* only run on deploy to v1 endpoints

* properly select subgraph-release

* file rename + refactor

* trigger ci.feature on any workflow edits given requirement of ci feature workflow for merge

Co-authored-by: elvijsTDL <[email protected]>
Co-authored-by: Miao ZhiCheng <[email protected]>
Co-authored-by: saflamini <[email protected]>
Co-authored-by: Joshua Trujillo <[email protected]>
Co-authored-by: Mehul Soni <[email protected]>
Co-authored-by: Lewington-pitsos <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Didi <[email protected]>
0xdavinchee added a commit that referenced this pull request May 24, 2022
* Add few more CFA and IDA cases (#781)

* SDK-Core and Subgraph Release Cleanup (#784)

* subgraph clean up
* make FlowOperatorUpdatedEvent immutable
* fix comment
* supertoken initialization wip
* added loadNativeAssetSuperToken, loadPureSuperToken and loadWrapperSuperToken functions in Framework
* loadSuperToken returns base SuperToken class
* added tests for new initialization
* move createFlowOperatorUpdatedEventEntity after flowOperator entity is created

* SDK-Core Version 0.4.0 (#782)

* changelog breaking added

* removed most of the readme file text

* update dependency versions

* update package.json dependency versions
* piggyback incorrect change-log versions
* update sdk-core changelog accordingly
* update sdk-core version in subgraph

* bump sdk-core version in sdk-redux

* update js-sdk version and yarn.lock

* Update CHANGELOG.md

* Update CHANGELOG.md

* update package versions for examples

* fix broken example build

* Update yarn.lock

* same as simple-acl-close

* forgot about yarn.lock file...

* fix transactiontable.tsx

Co-authored-by: Miao ZhiCheng <[email protected]>

* Refactor SuperfluidFrameworkDeployer & ERC1820RegistryCompiled (#789)

* refactor SuperfluidFrameworkDeployer & ERC1820 support
* include cfa/ida libs in the framework struct

* Simple ACL Gelato Example (#779)

* rename files
* fix forge build after copy pasta
* clean up Makefile
* local .gitmodules and remove .gitmodules in root
* comment out ida deployment as this is silently breaking
* add hardhat.config.ts
* create OpsMock contract
* setOps in resolver
* add tests for SimpleACLCloseResolver
* cleanup setup()
* setup cleanup
* test case cleanup
* remove superfluidtester file
* cleanup test file: remove comments, organize by section
* add more tests
* add more TODO tests
* add more functionality to simpleaclcloseresolver
* added a bunch of fuzzing tests because why not?
* added some general test cases
* add README.md
* add forge deploy in make file
* add check if flow exists for `canExec` boolean
* modify hardhat.config.ts to work with foundry test files
* fix tests accordingly (block.timestamp starts from 0)
* add hardhat-ethers, sdk-core and hardhat-etherscan
* hardhat deploy script added
* GAS_LIMIT added to .env.tempalte
* typechain-types added to .gitignore
* typechain added to hardhat config
* tsconfig added
* added scripts to package.json
* added verifyContract hardhat task
* modified README.md
* deploy script fleshed out
* remove gasLimit in resolver
* quick tests setup
* basic test setup
* build-and-test.sh script added
* package.json formatting
* deploy framework and tokens
* SimpleACLResolver tests added
* deploy script cleanup
* SimpleACLResolver.t.sol naming cleanup
* ethers required otherwise typings is weird
* add gas reporter/solidity coverage for hardhat

* Bump ethereum-contracts to 1.2.2 (#791)

* Employment Loan Example: Patch lend function (#786)

* Subgraph 1.3.2 patch (#794)

* I think this is leading to issues w/ subgraph breaking

* actual fix

* this was creating 0 address account token snapshots as it always saves

* update subgraph version

* test case added

ensure no Account or AccountTokenSnapshot entities with the zero address

* missing :

* query mistake

* cleanup + clarification

* allow zero address account/ATS

* incorrect test description

* Subgraph Fix Pt2 (#798)

* I think this is leading to issues w/ subgraph breaking

* actual fix

* this was creating 0 address account token snapshots as it always saves

* update subgraph version

* test case added

ensure no Account or AccountTokenSnapshot entities with the zero address

* missing :

* query mistake

* cleanup + clarification

* allow zero address account/ATS

* incorrect test description

* remove immutable: true test

* test theory

* Revert "test theory"

This reverts commit 5ae2ebd.

* add tests

* test pulling all the subgraph entities which have an `ILightEntity` and ensure that none of them break

* print out the retrieved data

* Update ACL example (#799)

* update ACL example

* update ethereum-contracts to v1.2.2
* utilize ERC1820RegistryCompiled
* fix SFDeployer to take new syntax
* fix Makefile commands

* use cfaLib from sfFramework

* use CFAv1Lib from SFDeployer

* Js-SDK maintenance branch release 0.6.0 (#802)

* Fix a query syntax in subgraph testing (#801)

* fix coverage test path issue

* fix ci: set working-directory correctly

* fix ci

* fix ci

* [skip ci] Add a NOTE to ci

* [778] SlotsBitmapLibrary Property Test Fuzzing (#790)

* CfAv1Library ACL support (#803)

* [RFR] Added logIndex property to events (#800)

* Fix codecov source files (#806)

* [RFR] 654 added a property to order events chronologically (#805)

* Fix ethereum-coverage code coverage  (#808)

* New package: hot-fuzz, a hot fuzz for testing Superfluid protocol and Super Apps (#708)

* Fix the flowlottery hot fuzz temp link (#812)

* add test to hardhat budget nft suite for transferring NFT ownership (#795)

Co-authored-by: Miao ZhiCheng <[email protected]>
Co-authored-by: 0xdavinchee <[email protected]>

* Some small changes (#813)

* docs: add bertux as a contributor for review (#820)

* Hotfuzz improvements (#814)

* update hot-fuzz version due to breaking change

* avalanche-c (#823)

* Avalanche Support Added (#819)

* avalanche support added

* avalanche support added for ethereum-contracts (hardhat), sdk-core in constants.ts and subgraph (config/, networks.json, addresses.template.ts)
* subgraph `README.md` cleanup
* deploy-all-networks variable renaming

* constants for chainId

* js-sdk getConfig update

* add avalanche to networkNameToChainIdMap

* goerli => matic rename

* canonical naming for avalanche-c

* added Avalanche to supertoken deployer Dapp

* readme update

* use canonical naming from truffle-config.js

* use truffle-config canonical names in constants for sdk-core

* fix empty pk case

* matic => polygon-mainnet for sdk-core tests

* new js-sdk/sdk-core versions bumped!

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

* [RFR] Added criticalAtTimestamp entity to subgraph v1 (#822)

* new field added maybeCriticalAtTimestamp account token snapshot
* Added maybeCriticalAtTimestamp account token snapshot
* added to SDK-Core subgraph files
* changed variable name to MAX_SAFE_MILLISECONDS

* [RFR] create account token snapshot log for each account token snapshot updates (#830)

* added logs entitt
* added code for account balance log
* fixed --  added test cases
* fixed ida broken test cases
* comment added
* merged critical at timestamp changes into branch
* added isLiquidationEstimateOptimistic flag
* updated:: as per review comment

* Create folder structure for examples (#804)

* Fixed:: TypeScript error in /Users/mikeghen/Documents/Projects/ricochet-frontend/node_modules/@superfluid-finance/js-sdk/src/InstantDistributionAgreementV1Helper.d.ts(197,19): (#829)

* allow same app registration on testnets and mainnets

* Sdkcore updates new fields and entities changes (#834)

* added account token balance logs and missing field from other story

* Revert "Sdkcore updates new fields and entities changes (#834)" (#836)

This reverts commit cd892eb.

* SDK-Core Release Subgraph Compatibility Tests (#835)

* add subgraph tests in sdk-core tests
* general ci cleanup: quote consistency, remove unused env variables
generate-graphql-schema for correct release in feature, canary and release
use hardhat node instead of truffle and update hardhat.config.ts to point to hardhat node chainId
separate startGanacheAndDeployContracts into startHardhatNode and setupTestEnvironment
* add multiple versions test to ci-pre-release
refactor 5_subgraph_test.ts by taking test functions out
new previous-versions-testing folder which contains queryTests.ts (extracted functions from above), runQueryTests.ts (for running previous-version-tests) and package.json which has sdk-core as a devDep
* generate-graphql-schema before build and release
* only test live subgraph release for sdk-core-release and only test previous sdk-version releases for subgraph release
* use query instead of framework for subgraph query tests + add console.log to see where things are breaking-local subgraph taking too long
* try reusable workflow, use query instead of full framework object
* test and deploy subgraph reusable workflow takes inputs, ci.canary, ci.feature, ci.pre-release cleanup w/ use of reusable workflow, use goerli TEMPORARILY because matic indexing is really slow
* rename call.setup-and-deploy to setup-deploy-and-test, cleanup + uses fix, testing https://github.blog/changelog/2022-01-25-github-actions-reusable-workflows-can-be-referenced-locally/, add test-subgraph-on-previous-sdk-versions to canary build and subgraph deploy, use local-subgraph-rule in test-previous-versions for deploy-subgraph case, LOCAL_SUBGRAPH_URL taken for runQueryTests
* check indexing completeness script added and reusable workflow added for it, deploy subgraph needs build and test w/ previous sdk-core releases to pass
* use matic instead of goerli
* workflow name headline consistency + double quote consistency cleanup + caller files added to dependent paths

* Fix workflow (#839)

* Yellow Paper Latex Pipeline

* Last CI fix  (#840)

* Local subgraph + SDK-Core tests fixes + cleanup

* Sdkcore updates new fields and entities changes (#837)

* Revert "Revert "Sdkcore updates new fields and entities changes (#834)" (#836)"

This reverts commit f8cb3ec.

* require test-subgraph-on-previous-sdk-versions

add test-subgraph-on-previous-sdk-versions to needs before publish-npm-packages job is run

* remove test-subgraph-on-previous-sdk-versions job from ci.canary

Co-authored-by: 0xdavinchee <[email protected]>

* CI Pre Release Draft Fix + CI Canary Cleanup (#843)

* cleanup + fix logic of build files

* generate dev schema + SUBGRAPH_RELEASE_TAG: dev

* only run on deploy to v1 endpoints

* properly select subgraph-release

* file rename + refactor

* trigger ci.feature on any workflow edits given requirement of ci feature workflow for merge

* double quote (#847)

* Re-write rewards-distribution-token using new sdk-core (#833)

Co-authored-by: elvijsTDL <[email protected]>
Co-authored-by: Miao ZhiCheng <[email protected]>
Co-authored-by: saflamini <[email protected]>
Co-authored-by: Joshua Trujillo <[email protected]>
Co-authored-by: Mehul Soni <[email protected]>
Co-authored-by: Lewington-pitsos <[email protected]>
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Didi <[email protected]>
Co-authored-by: @Ξthcode <[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.

[SUBGRAPH] Create AccountTokenSnapshotLog for each AccountTokenSnapshot update
3 participants