8000 feat: update ic-js for new nns did and last didc by peterpeterparker · Pull Request #1347 · dfinity/nns-dapp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: update ic-js for new nns did and last didc #1347

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

Merged
merged 6 commits into from
Sep 22, 2022

Conversation

peterpeterparker
Copy link
Member
@peterpeterparker peterpeterparker commented Sep 20, 2022

Motivation

Update ic-js libraries that contains the upgrade to recent version of didc and the did files for the last commit of the IC

PR

Changes

  • update nns-js, sns-js and utils-js
  • adapt neuron id conversion for new did array format

Copy link
Contributor
@lmuntaner lmuntaner left a comment

Choose a reason for hiding this comment

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

Just a comment that keeps me wondering how it was working before...

Already approving.

@peterpeterparker peterpeterparker enabled auto-merge (squash) September 22, 2022 05:35
@peterpeterparker peterpeterparker merged commit d6b9566 into main Sep 22, 2022
@peterpeterparker peterpeterparker deleted the feat/update-ic-js-didc branch September 22, 2022 05:42
peterpeterparker added a commit that referenced this pull request Sep 22, 2022
# Motivation

Follow-up of #1347 which was auto-merged even though tests were failing.

# Changes

- adapt tests for new candid files generated with last didc
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.

2 participants
0