Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

add ss58 id for nodle #7279

Merged
merged 2 commits into from
Oct 8, 2020
Merged

Conversation

ETeissonniere
Copy link
Contributor

@ETeissonniere ETeissonniere commented Oct 8, 2020

This PR modifies the list of reserved SS58 IDs to include one for Nodle (37, we will keep 42 for Arcadia, our tesnet).
We'd then modify our node to include the reserved ID once this PR is merged.

✄ -----------------------------------------------------------------------------

Thank you for your Pull Request!

Before you submitting, please check that:

  • You added a brief description of the PR, e.g.:
    • What does it do?
    • What important points reviewers should know?
    • Is there something left for follow-up PRs?
  • You labeled the PR appropriately if you have permissions to do so:
    • A* for PR status (one required)
    • B* for changelog (one required)
    • C* for release notes (exactly one required)
    • D* for various implications/requirements
    • Github's project assignment
  • You mentioned a related issue if this PR related to it, e.g. Fixes #228 or Related #1337.
  • You asked any particular reviewers to review. If you aren't sure, start with GH suggestions.
  • Your PR adheres to the style guide
    • In particular, mind the maximal line length of 100 (120 in exceptional circumstances).
    • There is no commented code checked in unless necessary.
    • Any panickers have a proof or removed.
  • You bumped the runtime version if there are breaking changes in the runtime.
  • You updated any rustdocs which may have changed
  • Has the PR altered the external API or interfaces used by Polkadot? Do you have the corresponding Polkadot PR ready?

Refer to the contributing guide for details.

After you've read this notice feel free to remove it.
Thank you!

✄ -----------------------------------------------------------------------------

@cla-bot-2020
Copy link

cla-bot-2020 bot commented Oct 8, 2020

@ETeissonniere it looks like you have not signed our contributor license aggreement yet. Please visit this link to sign our agreement. This pull request cannot be merged until the agrement is signed.

@cla-bot-2020
Copy link

cla-bot-2020 bot commented Oct 8, 2020

@ETeissonniere, Your signature has been received.

@bkchr
Copy link
Member

bkchr commented Oct 8, 2020

Please update the ss58registry.json file as well. Ty.

@ETeissonniere
Copy link
Contributor Author

Done! :)

@bkchr
Copy link
Member

bkchr commented Oct 8, 2020

@ETeissonniere can you please merge master. TY

@bkchr
Copy link
Member

bkchr commented Oct 8, 2020

bot merge

@ghost
Copy link

ghost commented Oct 8, 2020

Waiting for commit status.

@ghost
Copy link

ghost commented Oct 8, 2020

Checks failed; merge aborted.

@bkchr bkchr added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Oct 8, 2020
@bkchr bkchr merged commit 258558a into paritytech:master Oct 8, 2020
ordian added a commit that referenced this pull request Oct 9, 2020
…up-updates

* master:
  Async keystore + Authority-Discovery async/await (#7000)
  Fixes logging of target names with dashes (#7281)
  seal: Add automated weights for contract API calls (#7017)
  add ss58 id for nodle (#7279)
  Refactor CurrencyToVote (#6896)
  bump-allocator: document & poison (#7277)
  Reset flaming fir network (#7274)
  reschedule (#6860)
  Drop system cache for trie benchmarks (#7242)
  client: improve log formatting (#7272)
  Rework `InspectState` (#7271)
  SystemOrigin trait (#7226)
  Update ss58 registry for Dock network (#7263)
  .maintain/monitoring: Add alert when continuous task ends (#7250)
  Rename `TRANSACTION_VERSION` to `EXTRINSIC_VERSION` (#7258)
  Split block announce processing into two parts (#6958)
  Fix offchain election to respect the weight (#7215)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants