-
Notifications
You must be signed in to change notification settings - Fork 4
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
push to production-version 2024 #1539
Merged
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
* updated comment-card max width trying to fix issue 1480 * removed error for tests with no assertions
* changed saved match location to a preact-state * lint * changed from changeUrl to updateUrl removed the useState implementation for matchUrl and ran lint * removed test disable Co-authored-by: Caleb Eby <calebeby@users.noreply.github.com>
Co-authored-by: Elijah Vennebush <ev118@users.noreply.github.com> Co-authored-by: Alex <avhoi83@gmail.com>
Pushed this through quickly so I wouldn't have to deal with these errors on each individual branch before other changes were reviewed. No actual changes.
* created the alert * added a check that reports had been uploaded * grammar (yes I care) * fixed inconsistency
* created a dialog on editing EditableTexts by admins in user * refresh the user page after every change * switched to standard system of getting DB info for users * Revert "switched to standard system of getting DB info for users" This reverts commit 1c5a09e. * changed all self props to isCurrentUser * separated alertOnSaveAdmin function, small changes * changed selfEdit to isCurrentUser and removed all refetch changes
* Get vite to mostly work Co-authored-by: Elijah Vennebush <ev118@users.noreply.github.com> Co-authored-by: Alex <avhoi83@gmail.com> * Fix postcss config and delete more unused * Fix tests * Remove unused babel plugins * Adjust rollup config * Fix Service Worker and PWA stuff * Update to type:module in package.json * Undo needless change src -> @ * Undo more needless changes? * For now, undo linaria changes in all files * Remove unrelated change * Undo more changes * Remove more deps (will have to add back some of these) * Dep updates since last time I worked on this * Set up new version of linaria * Fix a silly CSS bug * Use port 2733 for preview as well * Fix TS imports * Increase min chunk size for chunk merging * Remove unmaintained postcss-font-magician * Update node LTS minimum * Replace Jest with Vitest for less config * Downgrade TS to avoid needing to update eslint-plugin-caleb in this PR * Remove unused old rollup config * Undo upgrading preact to avoid needing to fix type stuff in this PR * Re-update preact to fix integration with vite and prefresh * Fix chart where polished lib was included at runtime * Fix type errors * Use node 18 in CI * Delete commented code * Run prettier on html files too --------- Co-authored-by: Elijah Vennebush <ev118@users.noreply.github.com> Co-authored-by: Alex <avhoi83@gmail.com>
* updated year list on leaderboard to display correctly * added links to leaderboard cards * lint * Update package.json * Discard changes to package.json --------- Co-authored-by: Caleb Eby <caleb.eby01@gmail.com> Co-authored-by: Caleb Eby <calebeby@users.noreply.github.com>
* created about page and added link from sidebar menu * created the route * changed export to default * added more substance to about page * added text and styling * changed to grid layout and some editing * aesthetic changes * revisions to the about blurb * updated about page replaced grid with constrained widths added semantic HTML tags linked email address added header tag * fixed text width, header margins * padding and margin changes * editing and minor changes * replaced text with embedded expressions * adjusted width parameters to make text narrower * formatting changes * convention & hyperlink nits --------- Co-authored-by: Caleb Eby <calebeby@users.noreply.github.com> Co-authored-by: Caleb Eby <caleb.eby01@gmail.com>
Almost as good as the Vite PR! |
If we merge this PR as "rebase and merge", then it will not add a merge commit to the |
calebeby
approved these changes
Mar 5, 2024
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.
Changes since the last push are listed in the commits.