-
Notifications
You must be signed in to change notification settings - Fork 109
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
Add context/messaging to frequent rejection reasons #462
base: main
Are you sure you want to change the base?
Add context/messaging to frequent rejection reasons #462
Conversation
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.
This is amazing work @arelymartinez16!! There were a lot of different parts to this issue but you handled it all really well-just tested it locally and it's working great! I left a few minor comments, but this should soon be good to go 🚀
apps/admin-dashboard/app/routes/_dashboard.applications.$id._index.tsx
Outdated
Show resolved
Hide resolved
@@ -22,6 +23,8 @@ export function ApplicationRejectedEmail({ | |||
into our community at this time. | |||
</Email.Text> | |||
|
|||
<Email.Text>Rejection Reason: {rejectionReason}</Email.Text> | |||
|
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.
Note for @ramiAbdou: maybe we want to move the rejection reason elsewhere in the email? Also, one of the rejection reasons is currently "Incorrect or suspicious LinkedIn" which might be an unclear rejection reason to share.
@@ -0,0 +1,15 @@ | |||
import { type Kysely } from 'kysely'; |
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.
Note for @ramiAbdou: reminder to update the timestamp in file name
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! Rami will take a final look and we'll ship it 🚢
Description ✏️
Closes #216
Key Features:
Type of Change 🐞
Checklist ✅