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

[Para] Assigned Page - Sort #424

Closed
3 tasks
tam-janice opened this issue Sep 23, 2024 · 6 comments · Fixed by #459
Closed
3 tasks

[Para] Assigned Page - Sort #424

tam-janice opened this issue Sep 23, 2024 · 6 comments · Fixed by #459
Assignees

Comments

@tam-janice
Copy link

tam-janice commented Sep 23, 2024

As a para who has been assigned multiple benchmarks by the CM, I want to be able to sort my list of benchmarks so I can easily find the one I'm working on with a particular student

AC
Given a Para has logged in to their account
When they are on the 'assigned' page
They can sort using a dropdown

Given a Para has opened the dropdown
When they select 'Date assigned'
Then they can see the newest assigned benchmark at the top

Given a Para has opened the dropdown
When they select 'Student'
Then they can see the benchmarks alphabetical by student first name

Given a Para has logged in to their account
When they are on the 'assigned' page
The default sort is 'Student'

Link to design: https://www.figma.com/design/m09znscRXNqSziAiEbNLTf/Compass-Designs?node-id=14403-17081&t=NdiL4IQhZBJgYBKQ-1

Questions

Notes

  • We do not yet store the date that we assign a benchmark to a para. We could by default use the created_at date on a task, but IMO the date should be stored explicitly, and be in a new PR.
  • This will necessitate that we do not allow reassignment of a task by changing the assignee, only be creating a new task.
  • Add tests
@tam-janice tam-janice converted this from a draft issue Sep 23, 2024
@tam-janice
Copy link
Author

10/15/2024 Blocker
Couldn't figure out how to assign to a para and switch to a para as if we are logged in. This is related to issue #450

Workaround currently is to assign to an accoutn that is already logged in and then change the role of that signed in account to Para

@nickvisut
Copy link
Contributor

nickvisut commented Oct 16, 2024 via email

@canjalal
Copy link
Contributor

Vince to add tests

@canjalal canjalal linked a pull request Nov 4, 2024 that will close this issue
@canjalal canjalal removed a link to a pull request Nov 5, 2024
@canjalal
Copy link
Contributor

canjalal commented Nov 5, 2024

We do not yet store the date that we assign a benchmark to a para. We could by default use the created_at date on a task, but IMO the date should be stored explicitly, and be in a new PR.

This will necessitate that we do not allow reassignment of a task by changing the assignee, only be creating a new task.

@canjalal
Copy link
Contributor

canjalal commented Nov 5, 2024

Adding seeds for tasks, but the tasks's assignee id should be changed to an actual para's user_id once we are able to log in as an actual para and not just our first user.

@nickvisut nickvisut linked a pull request Nov 6, 2024 that will close this issue
@canjalal
Copy link
Contributor

canjalal commented Nov 6, 2024

Add frontend testing framework to the initial PR

@github-project-automation github-project-automation bot moved this from In Development to Done in Compass Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants