Remove hard-coded record types #110

Closed
roysc wants to merge 3 commits from roy/rm-hardcoded-records into main
Owner
  • Refactors the Record.Attributes from Any into a byte string and removes the hard-coded Protobuf record types.
  • Fixes EIP-712 bytes decoding.

Companion SDK PR: cerc-io/laconic-sdk#41

Resolves cerc-io/laconicd#107 and is initial work on cerc-io/laconicd#108

- Refactors the `Record.Attributes` from Any into a byte string and removes the hard-coded Protobuf record types. - Fixes EIP-712 bytes decoding. Companion SDK PR: https://git.vdb.to/cerc-io/laconic-sdk/pulls/41 Resolves https://git.vdb.to/cerc-io/laconicd/issues/107 and is initial work on https://git.vdb.to/cerc-io/laconicd/issues/108
roysc added 1 commit 2023-09-25 12:36:32 +00:00
roysc changed title from remove hard-coded record types, fix EIP-712 decoding to Remove hard-coded record types 2023-09-25 12:37:15 +00:00
roysc requested review from dboreham 2023-09-25 12:37:47 +00:00
roysc added 2 commits 2023-11-28 00:34:40 +00:00
this matches the change in follow up PR branch `graphql-to-ipld`
rm confusing comments
Some checks failed
Build / cleanup-runs (pull_request) Failing after 2s
Deploy Contract / cleanup-runs (pull_request) Failing after 3s
Dependency Review / dependency-review (pull_request) Successful in 1m1s
Deploy Contract / deploy (pull_request) Failing after 56s
Pull Request Labeler / triage (pull_request) Failing after 3s
Build / build (pull_request) Successful in 1m11s
Lint / Run flake8 on python integration tests (pull_request) Failing after 32s
Protobuf / lint (pull_request) Failing after 28s
Lint / Run golangci-lint (pull_request) Successful in 43s
Semgrep / Scan (pull_request) Failing after 11s
Tests / cleanup-runs (pull_request) Failing after 2s
Protobuf / break-check (pull_request) Failing after 20s
Run Gosec / Gosec (pull_request) Successful in 25s
Tests / test-importer (pull_request) Successful in 1m9s
Tests / test-unit-cover (pull_request) Successful in 1m10s
Tests / test-rpc (pull_request) Successful in 1m12s
CodeQL / Analyze (go) (pull_request) Successful in 3m41s
Tests / sdk_tests (pull_request) Failing after 1m40s
e173f0a321
Author
Owner

Also partial progress on cerc-io/laconicd#108

Also partial progress on https://git.vdb.to/cerc-io/laconicd/issues/108
Member

Superseded by cerc-io/laconicd#132

Superseded by https://git.vdb.to/cerc-io/laconicd/pulls/132
ashwin closed this pull request 2024-01-15 04:21:41 +00:00
Some checks failed
Build / cleanup-runs (pull_request) Failing after 2s
Deploy Contract / cleanup-runs (pull_request) Failing after 3s
Dependency Review / dependency-review (pull_request) Successful in 1m1s
Deploy Contract / deploy (pull_request) Failing after 56s
Pull Request Labeler / triage (pull_request) Failing after 3s
Build / build (pull_request) Successful in 1m11s
Lint / Run flake8 on python integration tests (pull_request) Failing after 32s
Protobuf / lint (pull_request) Failing after 28s
Lint / Run golangci-lint (pull_request) Successful in 43s
Semgrep / Scan (pull_request) Failing after 11s
Tests / cleanup-runs (pull_request) Failing after 2s
Protobuf / break-check (pull_request) Failing after 20s
Run Gosec / Gosec (pull_request) Successful in 25s
Tests / test-importer (pull_request) Successful in 1m9s
Tests / test-unit-cover (pull_request) Successful in 1m10s
Tests / test-rpc (pull_request) Successful in 1m12s
CodeQL / Analyze (go) (pull_request) Successful in 3m41s
Tests / sdk_tests (pull_request) Failing after 1m40s

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cerc-io/laconicd-deprecated#110
No description provided.