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] Implement booking history collections #721

Closed
7 tasks
choden-dev opened this issue Jul 30, 2024 · 0 comments · Fixed by #735
Closed
7 tasks

[BACKEND] Implement booking history collections #721

choden-dev opened this issue Jul 30, 2024 · 0 comments · Fixed by #735
Assignees
Labels
backend relating to code in the /server directory

Comments

@choden-dev
Copy link
Member

choden-dev commented Jul 30, 2024

Is your feature request related to a problem? Please describe.

I would like an admin view showing all bookings admin related history.
All administrator actions that happen in the bookings + availability tab
At the moment I cannot see what i've done, whos booking i’ve canceled and when ect.

You will need to create a history collection with the appropriate records - make sure the base interface is:

  • timestamp
  • change_type
    • deletion
    • availability
  • start_date
  • end_date

Based on the appropriate type extend the base type

Deletion

  • Need to have the uid associated with the deleted user

Availability

  • Store the slots (positive) made available or those made unavailable (negative)

After defining these in server\src\data-layer\models\firebase.ts

You will need to create a collection in server\src\data-layer\adapters\FirestoreCollections.ts

BEFORE MERGING

  • Integration test written for services
  • Schemas annotated if adding new models
  • Code generation run (hint: yarn workspace server tsoa spec-and-routes)
  • Appropriate mocks created where possible
  • PR Reviewed (For non-trivial changes)
  • Changes tested after rebasing on master or merging in master (hint: git fetch origin master:master, then git rebase master or git merge master)
  • All required PR checks passing
@choden-dev choden-dev added the backend relating to code in the /server directory label Jul 30, 2024
@choden-dev choden-dev self-assigned this Jul 31, 2024
@choden-dev choden-dev linked a pull request Aug 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend relating to code in the /server directory
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant