Skip to content
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

First draft feedback workshop #230

Merged
merged 7 commits into from
Nov 15, 2024

Conversation

fcaroline2020
Copy link
Contributor

No description provided.

Copy link

netlify bot commented Sep 15, 2024

👷 Deploy request for cyf-workshops pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 91d35e7

Copy link

netlify bot commented Sep 15, 2024

Deploy Preview for cyf-workshop ready!

Name Link
🔨 Latest commit 91d35e7
🔍 Latest deploy log https://app.netlify.com/sites/cyf-workshop/deploys/67378e65d6c7d700087664ba
😎 Deploy Preview https://deploy-preview-230--cyf-workshop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link
Member

@SallyMcGrath SallyMcGrath left a comment

Choose a reason for hiding this comment

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

I just saw you removed the review request so I stopped. Let me know when this is ready, and in the meantime mark it as draft?

Looking good so far. A bit ambitious on the amount of new information we can get through -- suggest setting some pre-work where people research the frameworks. You could set up a ChatGPT or Poe prompt for people to practise with using the different frameworks before they come to class.

Here's an example bot https://poe.com/CYF_Learn_Debugger

feedback/readme.md Outdated Show resolved Hide resolved
feedback/readme.md Show resolved Hide resolved
feedback/readme.md Outdated Show resolved Hide resolved
@fcaroline2020 fcaroline2020 marked this pull request as draft September 16, 2024 14:17
@fcaroline2020
Copy link
Contributor Author

I just saw you removed the review request so I stopped. Let me know when this is ready, and in the meantime mark it as draft?

Looking good so far. A bit ambitious on the amount of new information we can get through -- suggest setting some pre-work where people research the frameworks. You could set up a ChatGPT or Poe prompt for people to practise with using the different frameworks before they come to class.

Here's an example bot https://poe.com/CYF_Learn_Debugger

@SallyMcGrath I converted it to a draft because I wanted to discuss if this workshop makes sense to have since we already have two modules about giving/receiving feedback

  1. [PD] Asking for feedback (M2 S2)
  2. [PD] Giving feedback (M2 S3)

My opinion is that this is giving/receiving feedback is an important topic and can never be practiced enough but I also recognize there may be other more important PD topics.

feedback/readme.md Outdated Show resolved Hide resolved
Copy link

@kfklein15 kfklein15 left a comment

Choose a reason for hiding this comment

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

This looks great. Some small changes.

feedback/readme.md Outdated Show resolved Hide resolved
Add time estimates and finish introducing the feedback models
@fcaroline2020 fcaroline2020 marked this pull request as ready for review November 15, 2024 18:10
Copy link

@kfklein15 kfklein15 left a comment

Choose a reason for hiding this comment

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

Changes have been made. All good to go.

@fcaroline2020 fcaroline2020 dismissed SallyMcGrath’s stale review November 15, 2024 18:23

Reviewed by Karen, will pick up the chatbot idea during our next ITP sync to set up for Januarys cohort

@fcaroline2020 fcaroline2020 merged commit b7474c8 into CodeYourFuture:main Nov 15, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants