Skip to content

Conversation

@msquance-stem
Copy link
Contributor

@msquance-stem msquance-stem commented Mar 21, 2025

Status

Review progress:

  • Browser tested
  • Front-end review completed
  • Tech review completed

What's changed?

  • New button container component

Steps to perform after deploying to production

If the production environment requires any extra work after this PR has been deployed detail it here. This could be running a Rake task, migrating a DB table, or upgrading a Gem. That kind of thing.

@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2371 March 21, 2025 15:10 Inactive
@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2371 March 24, 2025 15:19 Inactive
@full_width = full_width
end

def column
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this be a bit more descriptive? "column_sizing" or something similar?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good shout!

Copy link
Contributor

@A-Wheeto A-Wheeto left a comment

Choose a reason for hiding this comment

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

Just one suggestion on the method naming in the component, if you think it's ok it doesn't need changing.

@msquance-stem msquance-stem temporarily deployed to teachcomputing-pr-2371 March 25, 2025 12:05 Inactive
@msquance-stem msquance-stem force-pushed the 2985-button-container-in-strapi branch from 322a1e9 to f32571a Compare April 2, 2025 11:24
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 2, 2025

@msquance-stem msquance-stem merged commit 73b893c into main Apr 2, 2025
5 checks passed
@msquance-stem msquance-stem deleted the 2985-button-container-in-strapi branch April 2, 2025 11:35
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.

4 participants