Chaos 752 review application page + 730 per port application summary - #757
Closed
peternuyn wants to merge 11 commits into
Closed
Chaos 752 review application page + 730 per port application summary#757peternuyn wants to merge 11 commits into
peternuyn wants to merge 11 commits into
Conversation
…ate when currentUser submit their rating.
* changed language mode from Django to TypeScript JSX
* feat: append placeholder variable to textarea
* feat: add {{offer_link}} button
* feat: add tooltip explanation on template buttons
* remove unnecessary css
* fix: add placeholder variable at cursor position
* fix: remove extra spaces in template button {{offer_link}} and make sure the base_url not get double slashed.
* fix: change replace to replaceAll
* add undo/redo functionality
* fix redo key shortcut
* move const table to summary page
* move select dropdown to summary page
* move out components
* basic table switch setup
* add colors to table
* add second table
* move send outcome email button into table component
* move 'all' and 'offered' tables into different components
* add 'sort by' select and 'start queue' button
* fix expand rows
* handle dynamic filter
* clean up logic
* fix getFilterCount
---------
Co-authored-by: Giselle Angela Hanjaya Putri <z5491414@ad.unsw.edu.au>
Co-authored-by: Peter Nguyen <z5662723@ad.unsw.edu.au>
…successful applicants
…rsion of this. Dead code so i delete.
… uses app.private_status, dont store these states (reject/pending/offer) into useState because this cache will be stale if there's an error during the changing of state.
peternuyn
changed the title
peternuyn
changed the title
Contributor
Author
|
giselle already added some extra functionalities in a new branch and merged my code in as well. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
I already link the Ui with the backend, so reccommend to manually test and lmk if there's any issue. Imma not merge this yet.