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

ID to address conversion fix #65

Merged
merged 2 commits into from
May 24, 2023
Merged

Conversation

amarinkovic
Copy link
Member

@amarinkovic amarinkovic commented May 23, 2023

When using a non-address ID, conversion throws an error. This is needed in start token sale flow. Workaround is provided with this PR.

@melks
Copy link

melks commented May 23, 2023

@amarinkovic amarinkovic force-pushed the CU-863gtbgy7_id_to_address_fix branch from f054c28 to 6cdd29e Compare May 23, 2023 11:43
Copy link
Contributor

@kevin-fruitful kevin-fruitful left a comment

Choose a reason for hiding this comment

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

lgtm ~

@amarinkovic amarinkovic changed the title ID to address conversion issue ID to address conversion fix May 24, 2023
@amarinkovic amarinkovic merged commit eecd850 into main May 24, 2023
@amarinkovic amarinkovic deleted the CU-863gtbgy7_id_to_address_fix branch May 24, 2023 07:40
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.

3 participants