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

Updated to polkadot latest version #241

Open

ayushmishra2005 wants to merge 20 commits into make_compatible_with_kusama from polkadotv0.9.11

Contributor

ayushmishra2005 commented Nov 11, 2021

No description provided.


          Updated to polkadot0.9.11

132fd48

ayushmishra2005 requested a review from ltfschoen

November 11, 2021 15:49

Ayush added 3 commits

December 1, 2021 21:35


          Fix extrinsic failure issue

401c1a0


          updated nightly version to fix CI issue

26f4bbd


          Fixed test-cases

b29e2db

ltfschoen reviewed

View reviewed changes

node/src/command.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

node/src/service.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/Cargo.toml Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/Cargo.toml Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/Cargo.toml Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs

+              impl WeightToFeePolynomial for WeightToFee {
+                  type Balance = Balance;
+                  fn polynomial() -> WeightToFeeCoefficients<Self::Balance> {
+                      // in Rococo, extrinsic base weight (smallest non-zero weight) is mapped to 1 MILLIUNIT:

Collaborator

ltfschoen Dec 9, 2021 •

edited

have you discussed these proposed values for p and q and WeightToFeeCoefficient field values with the team?

Contributor Author

ayushmishra2005 Dec 11, 2021

No @ltfschoen . I picked it from here https://github.com/substrate-developer-hub/substrate-parachain-template/blob/polkadot-v0.9.11/runtime/src/lib.rs#L139

Contributor Author

ayushmishra2005 Jan 4, 2022

I have added details in shared sheet Runtime Upgrade Values

Collaborator

ltfschoen Jan 5, 2022 •

edited

instead of storing them in an internal spreadsheet, could you please create a Github issue for each of the Runtime Upgrade Values that haven't been resolved.
ideally the comments associated with these Runtime Upgrade Values should be included as comments in the code so it appears in the documentation

Contributor Author

ayushmishra2005 Jan 6, 2022

raised here #246

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

runtime/src/lib.rs Outdated Show resolved Hide resolved

ltfschoen reviewed

View reviewed changes

node/src/chain_spec.rs Show resolved Hide resolved

ltfschoen requested changes

View reviewed changes

Collaborator

ltfschoen left a comment

i haven't tried running this, i've only reviewed the code and left comments.
if you think it's necessary for me to run it please share instructions


          Improve formatting

49adf4d

Collaborator

ltfschoen commented Dec 13, 2021

@ayushmishra2005 please DM me when you've responded to all the questions and made the linting changes so i can respond if necessary all at once

Ayush and others added 5 commits

December 14, 2021 18:27


          Improve formatting error

aa42f6a


          uprade version to 0.9.12


          Fixed compilation error

c942ce2


          changes to fix CI issue

fd40679


          Merge pull request #243 from DataHighway-DHX/polkadotv0.9.12

d09c96e

Polkadotv0.9.12

ayushmishra2005 changed the title ~~Updated to polkadot0.9.11~~ Updated to polkadot latest version

Ayush and others added 8 commits

January 2, 2022 09:40


          Upgraded to polkadot 0.9.13

9c366c6


          Merge pull request #244 from DataHighway-DHX/polkadotv0.9.13

dfd9627

Upgraded to polkadot 0.9.13


          removed unused variable

226f0e0


          make EXISTENTIAL_DEPOSIT similar to master


          made MILLISECS_PER_BLOCK similar to master

5af5dd6


          removed unused variable

47c1780


          removed unused variable

0d7e2e7


          made UncleGenerations similar to master

987a464

ltfschoen mentioned this pull request

parachain: Block production time for DataHighway parachain must match the relay chain #245

Open

Contributor Author

ayushmishra2005 commented Jan 10, 2022

i haven't tried running this, i've only reviewed the code and left comments. if you think it's necessary for me to run it please share instructions

@ltfschoen I have tested it locally. Its working fine. However if you want to test, you can follow https://docs.substrate.io/tutorials/v3/cumulus/start-relay/ and replace parachain-collator with datahighway-collator

Ayush added 2 commits

January 18, 2022 17:40


          Added polkadot-launch config

b1ef225


          changes to get deploy on Kusama

927b60e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment