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

Removed config proxy and config from deprecated example commands. #754

Merged
merged 33 commits into from
Nov 22, 2023

Conversation

cristure
Copy link
Contributor

Description of the pull request (what is new / what has changed)

Removed config proxy and config from deprecated example commands.

Did you test the changes locally ?

  • yes
  • no

Which category (categories) does this pull request belong to?

  • document new feature
  • update documentation that is not relevant anymore
  • add examples or more information about a component
  • fix grammar issues
  • other

schimih and others added 30 commits July 14, 2023 16:26
Update wallet section with Web-Wallet Guardians and xPortal Invisible Guardian
Fix URLs (web wallet provider)
Mention xSuite as an alternative of the now-deprecated erdjs-snippets.
Updated docusaurus to the latest version + Contract Calls
@cristure cristure self-assigned this Nov 21, 2023
@@ -19,11 +19,9 @@ In order to view the current configuration, one can issue the command `mxpy conf

```
{
"proxy": "https://gateway.multiversx.com",
"txVersion": "1",
Copy link
Contributor

Choose a reason for hiding this comment

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

This can be removed, as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

you mean the txVersion field?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes

Copy link
Contributor

@andreibancioiu andreibancioiu left a comment

Choose a reason for hiding this comment

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

🥳 🙌

Only the comment from @popenta.

@cristure cristure changed the base branch from main to development November 22, 2023 08:53
@cristure cristure merged commit 699187d into development Nov 22, 2023
1 check passed
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.

9 participants