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

[Refactoring] Need to ensure single default vault per campaign #420

Open
igoychev opened this issue Jan 23, 2023 · 2 comments
Open

[Refactoring] Need to ensure single default vault per campaign #420

igoychev opened this issue Jan 23, 2023 · 2 comments
Assignees

Comments

@igoychev
Copy link
Contributor

igoychev commented Jan 23, 2023

Which area(s) of Podkrepi.bg are affected? (leave empty if unsure)

Database

Describe the Situation

Currently the frontend does not properly support multiple vaults per campaign and it relies on that there is only one. However the backend allows it and this creates complexities of maintaining the correct logic.

The lack of UI support on the fronted also makes complex logic of calculating and displaying the reached, blocked and current amounts per campaign.

To Reproduce

On the frontend, when creating/editing a campaign there are no fields for creating/naming vaults and describing what they are for.

Expected Behavior

To stay safe until frontend starts supporting multiple vaults, the proposal is the backend to ensure a single vault named the same as the campaign title.

Which browser are you using? (if relevant)

No response

@igoychev igoychev added type: bug Something isn't working type: technical debt and removed type: bug Something isn't working labels Jan 23, 2023
@igoychev igoychev pinned this issue Feb 14, 2023
@yyosifov
Copy link
Contributor

Can I take this one?

@igoychev
Copy link
Contributor Author

Can I take this one?

Happy to see you taking it! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants