fix: Sort rubric objectives by creation date #90
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Save the objectives in an array and use an aux map to map objectives by their UUID with the array index
Merged
PedroChaparro
added a commit
that referenced
this pull request
Nov 16, 2023
* docs: Add bruno collection (#87) * docs: Create `CONTRIBUTING` file * docs: Add bruno collection * fix: Sort rubric objectives by creation date (#90) * chore(deps): Bump dependencies * chore: Bump golang version * chore: Update docker compose file to work with podman * fix(rubrics): Order rubric objectives by creation date * fix: Min criteria weight (#91) * feat: Update rubric objective (#93) * feat: Create endpoint to update an objective * test: Add test to ensure teachers can update rubric objectives * refactor(tests): Separate utils from test functions * docs(http): Update bruno collection * docs(http): Add insomnia collection * feat: Update objectives criteria (#94) * feat(rubrics): Create endpoint to update an objective criteria * docs(http): Update bruno collection * docs(http): Update insomnia collection * test(rubrics): Add test to ensure teachers can update objectives criteria * feat: Update rubric name (#95) * docs(openapi): Add new endpoint to specification * feat(rubrics): Create endpoint to update the rubric name * docs(http): Update bruno collection * docs(http): Update insomnia collection * test(rubrics): Add test to ensure teachers can update rubric names * feat: Remove objective (#98) * docs(openapi): Update specification Add new endpoints to delete rubric objectives and criteria * feat(db): Add on delete cascade to rubric objectives refs * feat(rubrics): Create endpoint to delete an objective * docs(http): Update bruno collection * docs(http): Update insomnia collection * test(rubrics): Add test to ensure teachers can delete rubric objectives * feat: Delete objective criteria (#99) * refactor(rubrics): Group rubrics DTOs * refactor(rubrics): Group rubrics requests * refactor(courses): Group courses DTOs * refactor(courses): Group courses requests * feat(db): Add on delete cascade constraint on criteria table references * feat(rubrics): Create endpoint to remove objectives criteria * fix(db): Increment max criteria weight decimals * test(rubrics): Add test to ensure teachers can delete criteria * docs(http): Update insomnia and bruno collections * ci: Trigger integration pipeline on pull request to `main`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes 📋
Related Issues 🔎
Closes #88