-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Adding the ability to add ideas / recommendations to a workshops block #848
Conversation
✅ Deploy Preview for cyf-programming ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for cyf-sdc canceled.
|
✅ Deploy Preview for cyf-curriculum ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
so let's instead pass through a map of related blocks
I've just removed that junk footer so we can see what we are doing as it's broken anyway
this would work for any block type you drop the partial call on to
feel free to make the nested block more fancy - it could be a slideshow, a panel viewer - whatever you feel like doing
probably also want to make all this skippable and well flagged to screenreaders
super useful to have this
this would work on all, but maybe let's just add if we find we need them
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.
ok, weird to self-review but this seems broadly fine and super useful in the way you have shown. Thank you SO much!
we need a new PR template for building new features so we also update documentation...
What does this change?
Adding the ability to add ideas / recommendations to a workshops block. An example of this being used is in the User Data / Sprint 2 / Day Plan.
Common Content?
Common Theme?
Adding a new workshop block with the ability to pass in workshop ideas.
Issue number: #792
Org Content?
Block Type
Checklist
Who needs to know about this?
@CodeYourFuture/itp-syllabus-team @SallyMcGrath