-
-
Notifications
You must be signed in to change notification settings - Fork 116
Fix #1066: Added time-stamper for Day Plan TOC timings #1623
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
base: main
Are you sure you want to change the base?
Fix #1066: Added time-stamper for Day Plan TOC timings #1623
Conversation
👷 Deploy request for cyf-curriculum pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-piscine pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-common pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-tracks pending review.Visit the deploys page to approve it
|
✅ Deploy Preview for cyf-sdc ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
👷 Deploy request for cyf-itd pending review.Visit the deploys page to approve it
|
👷 Deploy request for cyf-launch pending review.Visit the deploys page to approve it
|
|
Hi @CodeYourFuture, I’ve submitted this pull request (#1623) to fix the issue with the time-stamper for Day Plan TOC timings. Could you please review it when you get a chance? Your feedback and approval will be much appreciated so it can be merged. Thank you! Best regards, |
|
This looks like it breaks the display? In the deploy preview, I now see this:
But what I'm expecting to see is something like: How did you test this locally? How are you approaching trying to fix this? |

Added component to display start and end times in the Day Plan table of contents.