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

Replace CurrencySpace.axes with CurrencySpace.currencies in Quickstart #5

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

GrammAcc
Copy link
Owner

@GrammAcc GrammAcc commented Jan 3, 2024

I originally used the CurrencySpace.axes property in the quickstart tutorial for the valid options for <to_currency> and <from_currency>, but these should have been CurrencySpace.currencies for faster membership testing, so I have updated the quickstart tutorial to use CurrencySpace.currencies instead of axes.

I also added a direct link to the quickstart tutorial at the top of the README to make it more prominent for new users.

I originally used the CurrencySpace.axes property in the quickstart
tutorial for the valid options for <to_currency> and <from_currency>,
but these should have been CurrencySpace.currencies for faster
membership testing, so I have updated the quickstart tutorial to use
CurrencySpace.currencies instead of axes.

I also added a direct link to the quickstart tutorial at the top of the
README to make it more prominent for new users.
@GrammAcc GrammAcc merged commit 2028699 into main Jan 3, 2024
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.

1 participant