-
Notifications
You must be signed in to change notification settings - Fork 51
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
Genuary: include linked media from Genuary threads #821
Conversation
Need to fill out jobs Co-Authored-By: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Co-Authored-By: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
✅ Deploy Preview for acmcsuf ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
- cleaned up some commented out logs - linting - job logic Co-Authored-By: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
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.
Congratulations on your amazing PR, @jiink!
Excited to finally see .github/workflows/update_genuary.yaml
trigger in prod!
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.
Co-authored-by: Ethan Davidson <31261035+EthanThatOneKid@users.noreply.github.com>
Keep an eye out for https://acmcsuf.com/code/actions/workflows/update_genuary.yaml on March 29th, 2023! 👀 |
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.
Looks good!
To solve #769, the logic to get media from a pinned message needs to be changed.