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

Optimize USDC deployment flow #97

Merged
merged 4 commits into from
Feb 15, 2024
Merged

Optimize USDC deployment flow #97

merged 4 commits into from
Feb 15, 2024

Conversation

ygrishajev
Copy link
Contributor

@ygrishajev ygrishajev commented Feb 13, 2024

This PR is addressing issue #63.

  • fixes update propagation from the placement modal form to SDL page.
  • changes action button text from Close to Done which semantically suggests that modal changes are already applied. This seems the most optimal way atm as "save" functionality implementation would be a comparably much more effort
  • moves token select from placement modal to SDL root view

Updated SDL root view
image

Updated placement modal
image

@baktun14
Copy link
Contributor

This is great! 🚀

One last thing, there was a bug when changing the token it doesn't reflect in the generated SDL and transaction.

We call the function generateSdl here, it's somewhere in there.

@ygrishajev
Copy link
Contributor Author

ygrishajev commented Feb 14, 2024

One last thing

@baktun14 ok, spotted

@baktun14
Copy link
Contributor

Awesome, lgtm! I'll merge it later today.

@baktun14 baktun14 merged commit fad3169 into akash-network:main Feb 15, 2024
5 checks passed
@ygrishajev ygrishajev deleted the feature/optimize-usdc-deployment branch March 4, 2024 17:07
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.

2 participants