-
Notifications
You must be signed in to change notification settings - Fork 2
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
Merge Staging Commits into Main #540
base: main
Are you sure you want to change the base?
Conversation
* api and permsManager * enable eslint rule to check for promise use * added lint checking for promise misuse * fixed update-dev-db * disabled misused promise rule for api.ts * fixed import * fixed teamEvent typing issue * checks for document data * cast as document data * update candidateDecider * henry's comments --------- Co-authored-by: Henry Li <hl738@cornell.edu>
[diff-counting] Significant lines: 219. |
Found some weird issues going on while testing this out in the deployment preview: https://deploy-preview-540--cornelldti-idol.netlify.app/forms/shoutouts. Will check back on this later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
return ( | ||
<Item.Group divided> | ||
<Header className={styles.presentCount}> | ||
<Image className={styles.presentCountImg} src={catEmoji.src} alt="cat gif" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the header with the catEmoji
is showing in the ALL
view instead of PRESENT
view.
Summary
I branched off of staging and realized we still have about 5 commits here that never made it into main.
The only notable merge conflict was the following:
Refactor admin shoutouts (#463)
conflicted with
Display count for shoutouts (#462)
@oscarwang20 The second one is your PR. Can you verify that the functionality is all still there?
Notion/Figma Link
Test Plan
Notes
Breaking Changes