-
Notifications
You must be signed in to change notification settings - Fork 1
101 - Make calendar show ND In Tech one only (remove user's personal calendar) #102
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
Conversation
✅ Deploy Preview for ndit-old-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ndit-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
It looks like the prettier fail is because we're using a quite old version of node - I can replicate if I use node 14.17.6 locally, as we do in the pipeline. Any reason we want such an old version? Node 22 is current and will be maintained a while yet (https://nodejs.org/en/about/previous-releases). |
royemosby
left a comment
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 me ! 👉⏺️ ~~~~~~ 🚀
None that I'm aware of. We want to update that here or pull it out into another ticket? |
Made a PR here #103 |
Make calendar show ND In Tech one only
Description
Right now, the calendar on the site combines the ND In Tech calendar with the user's personal calendar. This removed code that shows the user's calendar.
Issue Board Ticket
Type of change
Change log
&<personal-cal-codefrom calendar embed URL.Testing
localhost:8080Checklist