Skip to content

Conversation

@anisharamnani
Copy link
Collaborator

@jenny-heath jenny-heath temporarily deployed to wrsat-wrsat-188-create--cvljwm January 15, 2026 21:36 Inactive
@anisharamnani anisharamnani temporarily deployed to wrsat-wrsat-188-create--cvljwm January 15, 2026 21:37 Inactive
} No newline at end of file
}

.notice {
Copy link
Collaborator

Choose a reason for hiding this comment

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

ah, so this affects the styling of all notices, which includes the one on the landing page that is white which we don't want to change. could you put these styles in a different class and apply it to the various places where we have these? you could do a similar thing to the notice on the landing page (line 29 of this file) and do .notice.teal

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes! that is a good idea & good call. next time i’ll do this i won’t assume that a style is being used in one place

Copy link
Collaborator

Choose a reason for hiding this comment

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

this test is fine to leave as is but for future reference we probably don't need to test every possible param value imo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ya that is fair…hmm…maybe i will just remove it then.

Copy link
Collaborator

@jenny-heath jenny-heath left a comment

Choose a reason for hiding this comment

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

only blocking comment is the styling thing!

@anisharamnani anisharamnani temporarily deployed to wrsat-wrsat-188-create--cvljwm January 20, 2026 22:35 Inactive
@anisharamnani anisharamnani reopened this Jan 21, 2026
@anisharamnani anisharamnani temporarily deployed to wrsat-wrsat-188-create--cvljwm January 21, 2026 15:08 Inactive
@jenny-heath jenny-heath temporarily deployed to wrsat-wrsat-188-create--yvqsrd January 21, 2026 15:10 Inactive
@anisharamnani anisharamnani temporarily deployed to wrsat-wrsat-188-create--yvqsrd January 21, 2026 15:11 Inactive
@anisharamnani anisharamnani merged commit 12b8bdc into main Jan 21, 2026
3 checks passed
@anisharamnani anisharamnani deleted the WRSAT-188-create-school-screen branch January 21, 2026 20:27
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