Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

feat(report): implement report api #2

Merged
merged 36 commits into from
Feb 13, 2024
Merged

feat(report): implement report api #2

merged 36 commits into from
Feb 13, 2024

Conversation

tomwwinter
Copy link
Contributor

No description provided.

@tomwwinter tomwwinter marked this pull request as ready for review February 8, 2024 05:38
@tomwwinter
ci: publish branch image
f28887e
@tomwwinter
ci: publish branch image
eddcbc4
@tomwwinter
ci: publish branch image
014db4d
@tomwwinter
ci: publish branch image
f9e2e8f
@tomwwinter
ci: publish branch image
9f20045
@tomwwinter
fix: bug fixes
e91aa7b
@tomwwinter
fix: set global prefix
aead16d
@tomwwinter
fix: null pointer
92fbeb1
@tomwwinter
fix: use user token in report-controller
da4c077
@tomwwinter
fix: error handling
0a37c7d
@tomwwinter
fix: remove global prefix
1b06625
@tomwwinter
fix: fix null pointer
12b81b6
@tomwwinter
fix: error handling
dad25c9
@tomwwinter
fix: permission check in controller
79f7573
@tomwwinter
Merge branch 'main' into tw/feat/report-api
a7299d9 
# Conflicts:
#	src/app.controller.spec.ts
#	src/app.controller.ts
@tomwwinter
fix: code clean up
3cd0503
@tomwwinter
Merge branch 'main' into tw/feat/report-api
df5de01 
# Conflicts:
#	src/app.controller.spec.ts
@tomwwinter
fix: deps clean up
92775f3
@tomwwinter
docs: remove outdated spec
cc75f35
@tomwwinter
docs: remove outdated docs
78a3ccf
@tomwwinter tomwwinter changed the title feat(report): add first report api draft feat(report): implement report api Feb 13, 2024
sleidig
sleidig reviewed Feb 13, 2024
View reviewed changes
Copy link
Member

@sleidig sleidig left a comment

Choose a reason for hiding this comment

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

Super clean and well structure. Nice! 😃

Let's extract the couchDb into a module (?) for re-use and then merge this PR as the first step.

.github/workflows/pr-update.yml Show resolved Hide resolved
nest-cli.json Show resolved Hide resolved
src/app.controller.ts Show resolved Hide resolved
src/query-body.dto.ts Show resolved Hide resolved
src/domain/reference.ts Show resolved Hide resolved
src/report/controller/dtos.ts Show resolved Hide resolved
src/report/storage/report-storage.service.ts Show resolved Hide resolved
src/report/repository/report-calculation-repository.service.ts Show resolved Hide resolved
src/report/repository/report-repository.service.ts Outdated Show resolved Hide resolved
src/report/repository/report-repository.service.ts Outdated Show resolved Hide resolved
@sleidig sleidig linked an issue Feb 13, 2024 that may be closed by this pull request
API Integration: implement report endpoints #7
Closed
@sleidig sleidig merged commit 055ea69 into main Feb 13, 2024
1 check passed
@sleidig sleidig deleted the tw/feat/report-api branch February 13, 2024 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@TheSlimvReal TheSlimvReal TheSlimvReal left review comments

@sleidig sleidig sleidig approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

API Integration: implement report endpoints
3 participants
@tomwwinter @sleidig @TheSlimvReal