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.

Sign up for GitHub

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

Jump to bottom

Fix N_ASSETS naming #200

Closed
enricobottazzi opened this issue Nov 30, 2023 · 0 comments · Fixed by #202
Closed

Fix N_ASSETS naming #200

enricobottazzi opened this issue Nov 30, 2023 · 0 comments · Fixed by #202
Assignees
@enricobottazzi
Milestone
V1 improvements a...

Comments

@enricobottazzi
Copy link
Member

enricobottazzi commented Nov 30, 2023 •
edited
Loading

Affects

contracts
zk_prover
backend

Proposals:

  • CURRENCIES_COUNT
  • N_CURRENCIES
  • BALANCE_COUNT

Also update summa-aggregation
@enricobottazzi enricobottazzi self-assigned this Dec 1, 2023
@enricobottazzi enricobottazzi moved this to In Progress in ∑ Summa Dec 1, 2023
@enricobottazzi enricobottazzi linked a pull request Dec 1, 2023 that will close this issue
Mantainance #202
Merged
This was referenced Dec 1, 2023
Merged
Closed
Closed
@github-project-automation github-project-automation bot moved this from In Progress to Merged in ∑ Summa Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@enricobottazzi enricobottazzi

Labels
None yet
Projects
∑ Summa
Status: Merged
Milestone
V1 improvements and consolidation
Development

Successfully merging a pull request may close this issue.

Mantainance summa-dev/summa-solvency
1 participant
@enricobottazzi