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

restrict user to 1 review per class #35

Closed
BlairCurrey opened this issue Oct 5, 2020 · 3 comments · Fixed by #42
Closed

restrict user to 1 review per class #35

BlairCurrey opened this issue Oct 5, 2020 · 3 comments · Fixed by #42
Assignees
@BlairCurrey
Projects
REPL Reviews v1

Comments

@BlairCurrey
Copy link
Collaborator

No description provided.

@BlairCurrey BlairCurrey created this issue from a note in REPL Reviews v1 (To do) Oct 5, 2020
@ankush
Copy link
Collaborator

ankush commented Oct 6, 2020

I think this will depend on #31 and #32

@BlairCurrey
Copy link
Collaborator Author

Agreed. #31 is now resolved.

Best we implement review updates first (#32).

After we implement the update feature, if a user tries to write a another review for the same class we can either flash a message indicating as much and requesting they update their current review (maybe with a relevant link to where they can update it), or redirect to the update form. Former is probably better IMO because it gives the user more context.

@BlairCurrey
Copy link
Collaborator Author

This is complete. I will link this issue to the PR for #32 when I make it.

@BlairCurrey BlairCurrey moved this from To do to In progress in REPL Reviews v1 Oct 10, 2020
@BlairCurrey BlairCurrey self-assigned this Oct 10, 2020
@BlairCurrey BlairCurrey mentioned this issue Oct 10, 2020
Merged
@BlairCurrey BlairCurrey linked a pull request Oct 10, 2020 that will close this issue
Update review #42
Merged
@BlairCurrey BlairCurrey moved this from In progress to Review in progress in REPL Reviews v1 Oct 12, 2020
REPL Reviews v1 automation moved this from Review in progress to Done Oct 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@BlairCurrey BlairCurrey

Labels
None yet
Projects
No open projects
REPL Reviews v1
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update review world-class/compass
2 participants
@ankush @BlairCurrey