Skip to content

restructure feedback workshop - #1953

Merged
Liam310 merged 4 commits into
mainfrom
feedback_workshop_streamline
Aug 4, 2026
Merged

restructure feedback workshop#1953
Liam310 merged 4 commits into
mainfrom
feedback_workshop_streamline

Conversation

@Liam310

@Liam310 Liam310 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

I really liked this workshop, but I thought it would benefit from a bit of a restructure. This PR aims to streamline the activity tasks by chopping some words and doing a bit of reformatting, and generally restructures the document slightly.

This involves:

  • Removing the "objective" section from each of the framework tasks - the objective is just to practise using the framework in question
  • Moving tips for feedback giving/receiving to the end as they make more sense as parting advice
  • Removing the stipulation to read a 1,800 word article before starting
  • Slight timing tweaks to include a discussion of framework choices
  • Adding a short conclusion
  • General formatting

Note: given that I've removed the expectation of reading the article at the start, do we think it ought to be added to prep/backlog?

@netlify

netlify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit 664efbb
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a71a12f3883890008cdd5a1
😎 Deploy Preview https://deploy-preview-1953--cyf-curriculum.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 88 (🟢 up 3 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit 664efbb
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6a71a12f33d25600098f7dae
😎 Deploy Preview https://deploy-preview-1953--cyf-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@illicitonion illicitonion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me!

I'd be pretty tempted to just drop down to one framework (personally I'd choose SBI) and spend more time talking about how to do that well and practicing, rather than having people use 4 frameworks - contrasting frameworks feels like it requires a lot more understanding/mastery of each, particularly if this is people's first experience of giving structured feedback.

Comment thread common-content/en/workshops/feedback/index.md Outdated
Comment thread common-content/en/workshops/feedback/index.md Outdated
@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 👀 In review in Curriculum Planning Jul 22, 2026
@Liam310

Liam310 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

I'd be pretty tempted to just drop down to one framework (personally I'd choose SBI) and spend more time talking about how to do that well

Interesting thought! I quite liked the workshop as an introduction to these ideas and a first go-around of attempting to apply them. Certainly they were not mastered but I suppose perhaps that wasn't the intended point maybe?

It could definitely be refactored into a more focussed workshop getting them to really practise and hone in on one (or perhaps two, I could see an argument for also including GROW as it is applicable to very distinct scenarios than the others). Do you want me to spend some time restructuring this workshop to focus specifically on a deeper dive in SBI? Or would you prefer to raise a ticket and to come back to it later (in either case, I probably won't be doing it imminently 😅 but someone else might!)

@illicitonion

Copy link
Copy Markdown
Member

I'd be pretty tempted to just drop down to one framework (personally I'd choose SBI) and spend more time talking about how to do that well

Interesting thought! I quite liked the workshop as an introduction to these ideas and a first go-around of attempting to apply them. Certainly they were not mastered but I suppose perhaps that wasn't the intended point maybe?

It could definitely be refactored into a more focussed workshop getting them to really practise and hone in on one (or perhaps two, I could see an argument for also including GROW as it is applicable to very distinct scenarios than the others). Do you want me to spend some time restructuring this workshop to focus specifically on a deeper dive in SBI? Or would you prefer to raise a ticket and to come back to it later (in either case, I probably won't be doing it imminently 😅 but someone else might!)

Happy either way :) If you agree narrowing a bit would make sense, let's at least raise a ticket

@Liam310
Liam310 merged commit 4069d97 into main Aug 4, 2026
11 checks passed
@Liam310
Liam310 deleted the feedback_workshop_streamline branch August 4, 2026 08:25
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Curriculum Planning Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants