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

feat: added getAllSchemaByDid method #8

Merged
merged 12 commits into from
Feb 8, 2024
Merged

feat: added getAllSchemaByDid method #8

merged 12 commits into from
Feb 8, 2024

Conversation

tipusinghaw
Copy link
Contributor

What

  • Added getAllSchemaByDid method
  • Removed did resolver direct implementation

@tipusinghaw
feat: implemented transaction estimation method
c9fc71e 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
fix: formatter issue
d548e18 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
fix: initialized the smart contract and rpcUrl
da18d81 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
fix: formatter issue
3550254 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
feat: implemneted logic for schema checksum
ef29450 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
refactor: added validation empty schema details
70ff9d5 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
feat: add getAllSchema method
d66ff61 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
Merge branch 'main' of https://github.com/ayanworks/polygon-schema-ma…
0e32dec 
…nager into POLYNDI-29-tx-fee-estimation

Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw
fix: lock.yaml
24feb07 
Signed-off-by: tipusinghaw <[email protected]>
@tipusinghaw tipusinghaw self-assigned this Feb 6, 2024
sairanjit
sairanjit requested changes Feb 6, 2024
View reviewed changes
src/schema-manager.ts Outdated Show resolved Hide resolved
sairanjit
sairanjit requested changes Feb 7, 2024
View reviewed changes
src/schema-manager.ts Outdated Show resolved Hide resolved
sairanjit
sairanjit approved these changes Feb 8, 2024
View reviewed changes
Copy link
Contributor

@sairanjit sairanjit left a comment

Choose a reason for hiding this comment

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

LGTM

@sairanjit sairanjit merged commit 505f81a into main Feb 8, 2024
3 checks passed
@sairanjit sairanjit deleted the POLYNDI-29-tx-fee-estimation branch February 8, 2024 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sairanjit sairanjit sairanjit approved these changes

Assignees

@tipusinghaw tipusinghaw

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tipusinghaw @sairanjit