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.

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

Chopsticks & E2E Tests #337

Merged
merged 4 commits into from
May 30, 2023
Merged

Chopsticks & E2E Tests #337

merged 4 commits into from
May 30, 2023

Conversation

gluneau
Copy link
Contributor

@gluneau gluneau commented May 3, 2023

Benefits of using chopsticks and examples of CLI

@gluneau gluneau requested a review from a team May 3, 2023 01:43
@gluneau gluneau requested a review from a team as a code owner May 3, 2023 01:43
@github-actions
Copy link

github-actions bot commented May 3, 2023

Visit the preview URL for this PR (updated for commit 9aef796):

https://astar-docs--pr337-chopsticks-fq56wcs5.web.app

(expires Tue, 30 May 2023 18:40:55 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f2f13e9b593d211faae6343d67a88fac3fd7268d

@gluneau gluneau added the documentation Improvements or additions to documentation label May 3, 2023
Copy link
Member

@andabak andabak left a comment

Choose a reason for hiding this comment

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

General comments:

  • You should cross link to Chopsticks documents for more reference (I did not notice this in the text)
  • also, based on Learning Session - commenting here but relates to he Astar config file on Chopstick repo - there should be some comments / explanation of Astar config and why it is set up the way it is (DOTs on Alice, USDT on Alice etc) (add as comments in that YAML file for sure)

@gluneau gluneau marked this pull request as draft May 12, 2023 13:28
@gluneau
Copy link
Contributor Author

gluneau commented May 12, 2023

Updates to comments upstream
AcalaNetwork/chopsticks#296

@gluneau gluneau marked this pull request as ready for review May 15, 2023 12:53
@gluneau gluneau requested a review from andabak May 16, 2023 14:53
@impelcrypto
Copy link
Member

It seems that the staging URL is not working on my end. @niklabh Do you know what might be the issue?
https://astar-docs--pr337-chopsticks-anp1w9k4.web.app/

image

@gluneau
Copy link
Contributor Author

gluneau commented May 23, 2023

@impelcrypto Previews dies of natural causes after a while. See the note it leaves in the preview link box

(expires Fri, 19 May 2023 23:15:42 GMT)

This can be used to look at the page content in the meanwhile:

https://github.com/AstarNetwork/astar-docs/blob/d692c24947bac953b13a984af269b7d775d09d8d/docs/build/environment/chopsticks.md

@sirius651 sirius651 self-requested a review May 30, 2023 06:45
@fiexer fiexer self-requested a review May 30, 2023 15:23
@fiexer fiexer dismissed andabak’s stale review May 30, 2023 15:24

Lets get it merged

@gluneau gluneau merged commit 5566cb0 into main May 30, 2023
3 checks passed
@gluneau gluneau deleted the chopsticks branch May 30, 2023 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants