Skip to content

Conversation

@popenta
Copy link
Contributor

@popenta popenta commented Jun 23, 2025

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

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

@popenta popenta self-assigned this Jun 23, 2025

#### Setting the explorer url

mxpy already knows the explorer urls for all three networks (Mainnet, Devnet, Testnet). This is particularly useful when running the CLI on custom networks where an explorer is also available. This key is not required to be present in the env config for the config to be valid. To set the explorer url use the following command:
Copy link
Contributor

Choose a reason for hiding this comment

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

mxpy and env can be somehow emphasized (e.g. code / italic / bold).

To delete an env config, we use the following command:

```sh
mxpy env remove <config-name>
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, now I see that we have both "env delete" and "env remove" 🙈

Might be tricky for users...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'd say it's pretty well explained what they both do.

Comment on lines 266 to 267
mxpy address set kind pem
mxpy address set path absolute/path/to/pem/wallet.pem
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry I did not see this before - how does this work? Only for the default address?

Copy link
Contributor

Choose a reason for hiding this comment

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

It's a bit tricky and ambiguous. Maybe we should let the users only adjust the properties while adding an alias, and not afterwards (that is, immutable entries).

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for only mentioning this now 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed as discussed.

As of October 2023, on Windows (or WSL), you might encounter some issues when trying to use Ledger in `mxpy`.
:::

## Interacting with the Multisig Smart Contract
Copy link
Contributor

Choose a reason for hiding this comment

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

🚀

--proxy=https://devnet-gateway.multiversx.com
```

## Interacting with the Governance Smart Contract
Copy link
Contributor

Choose a reason for hiding this comment

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

🚀

andreibancioiu
andreibancioiu previously approved these changes Jun 26, 2025

```sh
mxpy env set proxy_url https://devnet-api.multiversx.com
mxpy config-env set proxy_url https://devnet-api.multiversx.com --env devnet
Copy link
Contributor

Choose a reason for hiding this comment

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

For readability, the parameter --env sould have been before ... oh, nevermind, the previous ones are positional parameters.

danielailie
danielailie previously approved these changes Jun 26, 2025
@popenta popenta changed the base branch from main to development June 26, 2025 13:56
@popenta popenta dismissed stale reviews from danielailie and andreibancioiu June 26, 2025 13:56

The base branch was changed.

@popenta popenta merged commit ef64355 into development Jun 30, 2025
3 checks passed
@popenta popenta deleted the mxpy-examples-v11 branch June 30, 2025 07:27
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.

4 participants