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.

Sign up for GitHub

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

Jump to bottom

Issue 176 assign volunteers to repair request #237

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Issue 176 assign volunteers to repair request #237

wants to merge 33 commits into from

Conversation

ellenxxxiao
Copy link
Collaborator

@ellenxxxiao ellenxxxiao commented Feb 24, 2024 •
edited
Loading

Change Summary

  • Allow assigning the volunteer clicking on the assignee badge.
  • Once an assignee is assigned, change the status of the repair request to ACCEPTED.
  • A button to allow unassign a task, this will bring the status back to PENDING.

Change Form

  • The pull request title has an issue number
  • The change works by "Smoke testing" or quick testing
  • The change has tests
  • The change has documentation

ellenxxxiao and others added 27 commits January 20, 2024 14:30
@ellenxxxiao
initial commit
c8512dd
@Elefantat
Remove the ESLint warning
81e12a3
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
7c415c9 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
6a04d63 
…into issue-176-Assign_volunteers_to_repair_request
@Elefantat
Modify the style of volunteer card
62e4cdd
@Elefantat
Merge branch 'issue-176-Assign_volunteers_to_repair_request' of https…
6ad25ff 
…://github.com/codersforcauses/repair-labs into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
styling
c3b248b 
added skills display
addedd number of taskss assigned
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
ad57203 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
1. added endpoint for getting all repairers of a event
d28fb52 
2. styling
3. use endpoint to get data.
need STAFF table to proceed
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
7201d7e 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
- display userId on the badge
947486b 
- display the assigned volunteer as the first one in the list
@ellenxxxiao
allowed changing volunteer with badge
cb1f27c
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
c031540 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
deleted
484e979
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
a10735f 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
changes using new /events/[id]/repairers endpoint
658aab0
@ellenxxxiao
rmved logs
4720f94
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
654ed02 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
fix props requirement
349f134
@ellenxxxiao
Added accepted tasks count
931d625 
rmved unused comments
@ellenxxxiao
once a volunteer is assigned, change status to "ACCEPTED"
a357ff5
@ellenxxxiao
Merge branch 'issue-176-Assign_volunteers_to_repair_request' of https…
f5abe53 
…://github.com/codersforcauses/repair-lab into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
add button to unassign a task
f5141c4
@ellenxxxiao
added code to test
dade102
@ellenxxxiao
Merge branch 'main' into issue-176-Assign_volunteers_to_repair_request
e7373f4
@ellenxxxiao
fixed type errors
5218aa8
@ellenxxxiao
Merge branch 'issue-176-Assign_volunteers_to_repair_request' of https…
886b4cc 
…://github.com/codersforcauses/repair-lab into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao ellenxxxiao requested a review from dct0 February 24, 2024 05:43
justinliangg
justinliangg reviewed Feb 25, 2024
View reviewed changes
src/pages/api/repair-request/[id].ts Outdated Show resolved Hide resolved
src/schema/repair-request.ts Outdated Show resolved Hide resolved
src/pages/api/repair-request/[id].ts Show resolved Hide resolved
src/hooks/repair-request.ts Outdated Show resolved Hide resolved
src/hooks/events.ts Outdated Show resolved Hide resolved
src/components/Cards/assignee-badge.tsx Show resolved Hide resolved
justinliangg
justinliangg previously approved these changes Feb 25, 2024
View reviewed changes
src/components/Cards/assignee-badge.tsx Show resolved Hide resolved
src/components/Cards/assignee-badge.tsx Outdated Show resolved Hide resolved
ellenxxxiao and others added 6 commits February 27, 2024 23:53
@ellenxxxiao
Merge branch 'main' into issue-176-Assign_volunteers_to_repair_request
9ea29dd
@ellenxxxiao
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
45af26d 
…into issue-176-Assign_volunteers_to_repair_request
@ellenxxxiao
- removed isRepaired
841b251 
- fixed errors in hook
- refactored radio input field
- refactored assignee badge
@dct0
Merge branch 'main' of https://github.com/codersforcauses/repair-lab …
5355754 
…into issue-176-Assign_volunteers_to_repair_request
@dct0
Lint fix
2dda4db
@dct0
Rename repairStatus to status
439f942
@dct0
Copy link
Contributor

dct0 commented Apr 29, 2024

TODO: test frontend still works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@justinliangg justinliangg justinliangg left review comments

@dct0 dct0 Awaiting requested review from dct0

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
4 participants
@ellenxxxiao @dct0 @justinliangg @Elefantat