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

Backend support for the campaigns' expenses #453

Merged
merged 12 commits into from
Mar 29, 2023
Merged

Backend support for the campaigns' expenses #453

merged 12 commits into from
Mar 29, 2023

Conversation

slavcho
Copy link
Contributor

@slavcho slavcho commented Mar 12, 2023

We allow the user to add, fetch and delete campaign expenses.
Only the organizer (or admin) should be able to edit them.
Only an admin can approve an expense.
But the general public should be able to see any approved expenses.
The expense files are uploaded to AWS.

Environment

We need to create one new AWS bucket: 'expenses-files'

@github-actions
Copy link

github-actions bot commented Mar 12, 2023

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

Copy link
Contributor

@dimitur2204 dimitur2204 left a comment

Choose a reason for hiding this comment

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

Amazing job 🚀! Comments are regarding security and authentication mostly

apps/api/src/expenses/expenses.service.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.service.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Show resolved Hide resolved
apps/api/src/campaign/campaign.controller.ts Outdated Show resolved Hide resolved
@igoychev igoychev added the run tests Allows running the tests workflows for forked repos label Mar 17, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Mar 17, 2023
@igoychev igoychev added the run tests Allows running the tests workflows for forked repos label Mar 17, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Mar 17, 2023
apps/api/src/expenses/expenses.controller.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Outdated Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Show resolved Hide resolved
apps/api/src/expenses/expenses.controller.ts Outdated Show resolved Hide resolved
@igoychev igoychev added the run tests Allows running the tests workflows for forked repos label Mar 25, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Mar 25, 2023
slavcho added 5 commits March 26, 2023 14:54
1. remove can-edit endoint - we can do this in the frontent with the current endpoints.
2. move list expenses to the campaign controller.
3. Fetch the uploader it from the logged user. It used to be hardcoded.
@igoychev igoychev added the run tests Allows running the tests workflows for forked repos label Mar 29, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Mar 29, 2023
@igoychev igoychev merged commit f499978 into podkrepi-bg:master Mar 29, 2023
@igoychev igoychev mentioned this pull request Apr 3, 2023
8 tasks
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