Create shareable links for all seats, even if student is not assigned#87
Create shareable links for all seats, even if student is not assigned#87abigailrb03 wants to merge 3 commits intomainfrom
Conversation
Coverage for unit tests for Python 3.10
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Coverage for unit tests for Python 3.12
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #87 +/- ##
=======================================
Coverage 46.08% 46.08%
=======================================
Files 28 28
Lines 2018 2018
=======================================
Hits 930 930
Misses 1088 1088 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Coverage for unit tests for Python 3.11
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Coverage for unit tests for Python 3.10
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Coverage for unit tests for Python 3.12
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Coverage for e2e tests for Python 3.11
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Coverage for e2e tests for Python 3.10
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@phrdang does the current fix meet the use case you had envisioned or would a different approach be more helpful? perhaps a more user-friendly solution might follow something like this (very) low-fi drawing (though this will probably require rewriting some tests...) |
phrdang
left a comment
There was a problem hiding this comment.
It's a bit difficult for me to properly review this since I'm not familiar with the seating tool code, but what you changed seems to make sense to me on paper. I might be helpful if you could do a screen recording of the flow in addition to the screenshots you already sent? (And in the screen recording somehow copy and paste the link in a new tab to show that it works?)
| <img class="photo" src="{{ url_for('student_photo', exam_student=(room.exam, seat.assignment.student)) }}" alt="photo for: {{ seat.assignment.student.name }}"/> | ||
| <br>{{ seat.assignment.student.sid }} | ||
| {% endif %} | ||
| {% endif %} -m |
Yeah that UI could be useful in case a staff member just wants to copy the URL and not go to the page for that seat (saves a few clicks). |
|
Also thanks for working on this! |


resolving issue #73 by opening shareable link in new tab whenever a seat is clicked on. See images below:
