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

Invalidate a successfull donation #597

Merged
merged 10 commits into from
Jan 25, 2024
Merged

Invalidate a successfull donation #597

merged 10 commits into from
Jan 25, 2024

Conversation

slavcho
Copy link
Contributor

@slavcho slavcho commented Jan 16, 2024

This invalidates a successful donation and removes the money from the vault.

Motivation and context

In some rare cases we have a duplicated or an invalid donation from stripe.
We want to be able to manual adjust this from the admin panel.

We would need to add a new role for that in keycloak: account-edit-financials-requests

Testing

Provided a unit test and tested manually.

slavcho and others added 7 commits November 1, 2023 22:00
The idea of the cache is to help in extreme scenarios when many requests are being fired.
One request every 2 seconds should be easy to handle by the backend.
…to upload files with cyrilic names. But it adds a bit of complexity in the backend.
…ggy stripe donation and we would like to sort of remove it.

This is allowed only if one has special credentials, of course.
Copy link

github-actions bot commented Jan 16, 2024

✅ Tests will run for this PR. Once they succeed it can be merged.

@slavcho slavcho added the run tests Allows running the tests workflows for forked repos label Jan 16, 2024
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Jan 16, 2024
@sashko9807 sashko9807 added the run tests Allows running the tests workflows for forked repos label Jan 25, 2024
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Jan 25, 2024
@slavcho slavcho merged commit c0e7cf4 into podkrepi-bg:master Jan 25, 2024
9 of 10 checks passed
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