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

Giveaway Modal Release! #947

Merged
merged 3 commits into from
Oct 18, 2024
Merged

Giveaway Modal Release! #947

merged 3 commits into from
Oct 18, 2024

Conversation

nidhi-mylavarapu
Copy link
Contributor

Summary

This pull request is to release our Giveaway Modal, ending October 31st at 11:59 PM!

Remaining TODOs:

  • Fix Cypress Tests :)

Destaq and others added 3 commits October 2, 2024 17:48
* chore: normalize search

* feat: support longer searches by code *and* title
* giveaway modal

* new files

* functionality + migration

* type check

* frontend fix

* frontend fix

* add giveaway cutoff date

* temporary 10/5 date for testing

* syntax fix

* testing new date

* added giveaway cutoff date, tested

* updated vite

* reverted commit

* changed version

* attempting to address frontend cypress test error

* changed test

* accessibility test and hidden feature test removed

* added back to accessibility spec, added in new exit location for giveaway modal

* added specifics on giveaway modal accessibility test

* removed unnecessary frontend giveaway tests

* frontend fixes

* changed artifact version back to v2

* artifict version v3

* temporarily removed CUReviews Test

* fixed CUreviews api, uncommented CUReviews test

---------

Co-authored-by: Nidhi Mylavarapu <nidhi.mylavarapu@gmail.com>
@nidhi-mylavarapu nidhi-mylavarapu requested a review from a team as a code owner October 18, 2024 21:57
@dti-github-bot
Copy link
Member

[diff-counting] Significant lines: 342.

Copy link
Contributor

Visit the preview URL for this PR (updated for commit 9332b4f):

https://cornelldti-courseplan-dev--pr947-main-w4z6mqyp.web.app

(expires Sun, 17 Nov 2024 21:58:34 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6d4e0437c4559ed895272bbd63991394f1e0e933

Copy link
Contributor

@plumshum plumshum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change in CU Review API to fix some cypress tests
manually changed seenGiveAwayModal to pass rest of the cypress tests, will fix later
letsssss goooooooooooooooooooooooooooooooooooooooooooooo

Copy link
Member

@Destaq Destaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Collaborator

@nit0107 nit0107 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

YAYYY Giveaway Modal!!!

@nidhi-mylavarapu nidhi-mylavarapu merged commit 12c1f1f into release Oct 18, 2024
12 checks passed
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.

6 participants