Frankreed/unify result screens #96
Merged
Conversation
This file contains 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
Gehrkej
reviewed
Apr 5, 2024
Gehrkej
added a commit
that referenced
this pull request
Apr 12, 2024
* Added handling for non numbers and empty values * Added comments for future work * Frankreed/unify result screens (#96) * Result screens unified! * fixed the display bug in result * added title to styles, some redundancy removed from barChartScreen --------- Co-authored-by: Jake Gehrke <gehrkej@oregonstate.edu> * Planscreen player (#131) * plan hooked fully to database and finished by submitting drill * scroll down to refresh * change naming to be more clear and requre drill id and assigned_time * add assigned_data to user on user creation * fix key issue, lighter green, squircle, spacing of description * styling * updated prettier and github action settings to be more strict * code cleanup * ran pretty * attemptRefId into attemptId --------- Co-authored-by: Frankreed <baoanhtdn@gmail.com> * added hard coded helper text * helperText is now pulled from database * Added handling for non numbers and empty values * Added comments for future work * added hard coded helper text * helperText is now pulled from database * added error handling for submit drill button * adjusted uploadAttempt function after rebase * updated SID to be 1-20 * added error handling for submit drill button * adjusted uploadAttempt function after rebase * updated SID to be 1-20 * Inital changes requested from frank in PR * Converted error banners to error dialogs * Frankreed/Bar chart is 60fps consistent now (#129) * Database updated and codebase uses title instead of drillType * Removed title and added prettyDrillType and subType * Added putting to database and updated codebase * Rebased with layout and prettiier ran * leaderboard is more resilient to incorrect data * style changes * barChart performance vastly improved. Frame rate is now pegged at 60 (for ios). Still crashes on Android. * this doesn't improve anything measurable, it just makes sense in my head * Paginate barChart * very minor tick change/add back * barChart now displays 0 as the minimum instead of the lowest value (if the lowest value is lower than 0, display that instead). Illegal attempts shall be terminated --------- Co-authored-by: hannacol <hannacol@oregonstate.edu> * Minor cosmetic fixes (#138) * Database updated and codebase uses title instead of drillType * Removed title and added prettyDrillType and subType * Added putting to database and updated codebase * Rebased with layout and prettiier ran * made target an array for all drill types for consistency and fixed shotAccordion shotNum display * updated more code for target lists * Result screen works for putting drills * style changes * Ran prettier * Updated Headers to hold title better * prettier ran * ran prettier * added margin to the left and right of the segmented buttons * commented out a lot of prints, changed uid to a string instead of reference, sorted user in Teams based on role * Fixed shotAccordion's weird shot spacing --------- Co-authored-by: hannacol <hannacol@oregonstate.edu> * Database updated and codebase uses title instead of drillType (#134) * Database updated and codebase uses title instead of drillType * Removed title and added prettyDrillType and subType * Added putting to database and updated codebase * Rebased with layout and prettiier ran * made target an array for all drill types for consistency and fixed shotAccordion shotNum display * Result screen works for putting drills * style changes * ran prettier * Made attemptShots use an object instead of an array for targets and componentized header * Fixed drill list styling and ran prettier after rebase --------- Co-authored-by: Frankreed <baoanhtdn@gmail.com> * Reworking Drill Screens to be Componentized (#140) * DrillList has been componentized for main drill list and profile view to be sectioned off * Moved glyphs to be in section header instead of drill card * Drlll Description componentized for description screen and description modal and deleted image carousel * ran yarn pretty * slight optimization to getUnique * Removed prettyDrillType and modified drillType --------- Co-authored-by: Frankreed <baoanhtdn@gmail.com> * ErrorDialog Component Created and Styled * Added styling to Leave Dialog * Swapped order of action buttons in leave drill dialog * Created submitVisable varible * Added handling for non numbers and empty values * Added comments for future work * added hard coded helper text * helperText is now pulled from database * added error handling for submit drill button * adjusted uploadAttempt function after rebase * updated SID to be 1-20 * Added handling for non numbers and empty values * added error handling for submit drill button * Inital changes requested from frank in PR * Converted error banners to error dialogs * ErrorDialog Component Created and Styled * Added styling to Leave Dialog * Swapped order of action buttons in leave drill dialog * Created submitVisable varible * added header import back * fix weird stuff with rebase * more weird rebase things * Restart Drill functionality working (#136) * fixed duplicate sid * All dialogs using DialogComponent * Added handling for non numbers and empty values * Added comments for future work * added hard coded helper text * helperText is now pulled from database * added error handling for submit drill button * adjusted uploadAttempt function after rebase * updated SID to be 1-20 * Added handling for non numbers and empty values * added error handling for submit drill button * Inital changes requested from frank in PR * Converted error banners to error dialogs * ErrorDialog Component Created and Styled * Added styling to Leave Dialog * Swapped order of action buttons in leave drill dialog * Created submitVisable varible * added header import back * Added handling for non numbers and empty values * Added comments for future work * adjusted uploadAttempt function after rebase * Added handling for non numbers and empty values * ErrorDialog Component Created and Styled * All dialogs using DialogComponent * Fixed uploadAttempt header comment * changed == to ===, and added one missing import, remove unused imports, and fixed calculateProxHole cuz I was stupid... * fixed faulty getUnique --------- Co-authored-by: Frank Nguyen <41023671+FrankreedX@users.noreply.github.com> Co-authored-by: ajpert <84763013+ajpert@users.noreply.github.com> Co-authored-by: Frankreed <baoanhtdn@gmail.com> Co-authored-by: hannacol <hannacol@oregonstate.edu> Co-authored-by: hannacol <91215417+hannacol@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Result screens are now merged