Skip to content
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

achievement page #1005

Merged
merged 3 commits into from
Oct 3, 2023
Merged

achievement page #1005

merged 3 commits into from
Oct 3, 2023

Conversation

k2xl
Copy link
Collaborator

@k2xl k2xl commented Sep 29, 2023

No description provided.

@k2xl k2xl requested a review from sspenst September 29, 2023 19:15
@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Attention: 32 lines in your changes are missing coverage. Please review.

Comparison is base (22934e8) 85.01% compared to head (5450fad) 84.51%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1005      +/-   ##
==========================================
- Coverage   85.01%   84.51%   -0.50%     
==========================================
  Files         195      196       +1     
  Lines        5438     5470      +32     
  Branches     1168     1172       +4     
==========================================
  Hits         4623     4623              
- Misses        782      813      +31     
- Partials       33       34       +1     
Files Coverage Δ
pages/achievement/[type].tsx 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sspenst sspenst merged commit 4699346 into main Oct 3, 2023
@sspenst sspenst deleted the achievementpage branch October 3, 2023 03:37
sspenst added a commit that referenced this pull request Oct 4, 2023
* i forgot i had made this page on a branch when we did achievements

* tweak

---------

Co-authored-by: Spencer Spenst <[email protected]>
sspenst added a commit that referenced this pull request Oct 4, 2023
* next-themes wip

* signup toast and game.tsx change

* nuke newrelic frontend because possible memory issue on windows

* update git ignore, add public/sounds/music

* /campaign perf. From ~1.1s load to about 200ms

* fix test

* cleanup

* reset leastMoves on unpublish

* post level modal and tooltips (#1000)

* post level modal and tooltips

* wip

* revert collection-by-id

* updates

* delete capitalized file

* PR todos done

* cleanup

* fix build

* useeffect for query params

* tweaks

* simplify

---------

Co-authored-by: Spencer Spenst <[email protected]>

* ch4 preview (#1006)

* collection-by-id aggregate (#1003)

* wip

* fix issue with collection ordering

* fix test

* improve getCollection across website

* improving agg for edit levels

* rename

* improve profile collections loading by ~25%

* fix some issues with getCollections

* fix tests

* fix bug found from broken test.

* knock another 100ms off profile load

* 👍

* forgot 1 file

---------

Co-authored-by: Danny Miller <[email protected]>

* postgamemodal improvements

* more intuitive campaign button

* login link for some tips

* [Snyk] Upgrade pureimage from 0.3.17 to 0.4.7 (#977)

* fix: upgrade pureimage from 0.3.17 to 0.4.7

Snyk has created this PR to upgrade pureimage from 0.3.17 to 0.4.7.

See this package in npm:
https://www.npmjs.com/package/pureimage

See this project in Snyk:
https://app.snyk.io/org/k2xl/project/cf09253a-8e4c-4f7f-b75f-a1763e5c78eb?utm_source=github&utm_medium=referral&page=upgrade-pr

* pureimage type declaration

* fix

---------

Co-authored-by: snyk-bot <[email protected]>
Co-authored-by: Spencer Spenst <[email protected]>

* add reviews and 'dont show this popup for 24h' checkbox

* getRecommendedLevel dont show a level they have played recently

* get rid of datetime filter

* Tweak. Remove unnecessary colon

* complete/incomplete filter button for chapters

* indicators for marking ready

* tooltip for checkpoint

* cleanup

* disable useswr unless popup is open

* share icons on post modal

* sharebar

* curator edit/delete reviews (#1009)

* curator edit/delete reviews

* red ellipsis

* test

* fix flaky tests

* merge

* clear review rating button

* curator level management ux

* delete achievements on user delete

* use level userId instead of req userId

* move add to/edit level out of header dropdown

* hotfix for deleted reviews on home page

* achievement page (#1005)

* i forgot i had made this page on a branch when we did achievements

* tweak

---------

Co-authored-by: Spencer Spenst <[email protected]>

* update calc_records on archive

* Revert "update calc_records on archive"

This reverts commit 06f89c8.

* Try to get twitter images to show

* change to twitter site for card...

* apparently twitter now requires full url for og image

* also do ogurl just in case

* mobile buttons update (#1011)

* mobile buttons update

* optimizations

* mobile tweaks

---------

Co-authored-by: Spencer Spenst <[email protected]>

* integrityCheckRecords

* more accurate preventKeyDownEvent

* multiSelectLevel (from #1008)

* handle app theme in theme modal

* use dark class condition instead of checking theme

---------

Co-authored-by: Danny Miller <[email protected]>
Co-authored-by: snyk-bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants