Skip to content

restructure feedback workshop - #1953

Open
Liam310 wants to merge 1 commit into
mainfrom
feedback_workshop_streamline
Open

restructure feedback workshop#1953
Liam310 wants to merge 1 commit 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 07ad936
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a60e29d2c30d7000848e0de
😎 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: 89 (🟢 up 4 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 07ad936
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6a60e29d5f97350008f09ded
😎 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.


After the workshop today, participants will be able to:

- [ ] Choose a framework for giving clear, direct feedback depending on the scenario.

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.

Let's move these LOs up to the objectives key in the frontmatter

Comment on lines +58 to +60
- Situation: Describe the situation
- Behavior: Describe the behavior observed
- Impact: Explain the effect of the behavior on you, your team, or the organization

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.

Suggested change
- Situation: Describe the situation
- Behavior: Describe the behavior observed
- Impact: Explain the effect of the behavior on you, your team, or the organization
- **Situation**: Describe the situation. This should be objective - no personal interpretations or feelings.
- **Behavior**: Describe the behavior observed. This should also be objective.
- **Impact**: Explain the effect of the behavior on you, your team, or the organization. This can be subjective - it's ok to talk about how things made you feel. But don't guess _why_ someone else did something.

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

2 participants