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

post level modal and tooltips #1000

Merged
merged 14 commits into from
Sep 29, 2023
Merged

post level modal and tooltips #1000

merged 14 commits into from
Sep 29, 2023

Conversation

k2xl
Copy link
Collaborator

@k2xl k2xl commented Sep 27, 2023

TODO

  • next level link when in a collection does not keep URL params
  • clicking the suggested level should close the modal

@k2xl k2xl requested a review from sspenst September 27, 2023 15:16
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (746aa8c) 85.01% compared to head (43e0f70) 85.01%.

❗ Current head 43e0f70 differs from pull request most recent head fa16180. Consider uploading reports for the commit fa16180 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1000   +/-   ##
=======================================
  Coverage   85.01%   85.01%           
=======================================
  Files         195      195           
  Lines        5439     5439           
  Branches     1168     1168           
=======================================
  Hits         4624     4624           
  Misses        782      782           
  Partials       33       33           

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

@sspenst sspenst merged commit 58800b8 into main Sep 29, 2023
@sspenst sspenst deleted the postlevelmodal branch September 29, 2023 21:38
sspenst added a commit that referenced this pull request Sep 30, 2023
* 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]>
sspenst added a commit that referenced this pull request Oct 4, 2023
* 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]>
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