Skip to content

Conversation

@sra405
Copy link
Contributor

@sra405 sra405 commented Jul 24, 2024

What's changed?

  • code has been added to the school show response for the dashboard
  • project has been added to the lesson index so we can direct students to the project from the class page

@cla-bot cla-bot bot added the cla-signed label Jul 24, 2024
@raspberrypiherokubot raspberrypiherokubot temporarily deployed to editor-api-p-add-proper-itbt52 July 24, 2024 09:04 Inactive
@sra405 sra405 changed the title Add-properties-to-existing-responses Add properties to existing responses Jul 24, 2024
Comment on lines +20 to +26
if lesson.project
json.project(
lesson.project,
:identifier,
:project_type
)
end
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we add a test for this? No worries though if it's going to get complex 😅

@sra405 sra405 temporarily deployed to editor-api-p-add-proper-itbt52 July 25, 2024 14:41 Inactive
@sra405 sra405 temporarily deployed to editor-api-p-add-proper-itbt52 July 25, 2024 14:54 Inactive
Copy link
Contributor

@loiswells97 loiswells97 left a comment

Choose a reason for hiding this comment

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

Looks great, let's get it out there! 🚀

@sra405 sra405 merged commit 34a5c63 into main Jul 25, 2024
@sra405 sra405 deleted the add-properties-to-existing-responses branch July 25, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants