Skip to content

Conversation

@stopfstedt
Copy link
Member

bolshoi refactoring and kruft killing of table-specific stylesheets that were consuming the mixin.

fixes ilios/ilios#6611

refs ilios/ilios#6608

@netlify
Copy link

netlify bot commented Nov 5, 2025

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit 67a2a5c
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/690d7e3609ab0100082fb8f6
😎 Deploy Preview https://deploy-preview-8926--ilios-frontend.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 project configuration.

@stopfstedt stopfstedt marked this pull request as ready for review November 5, 2025 19:36
@stopfstedt stopfstedt marked this pull request as draft November 5, 2025 19:37
@stopfstedt stopfstedt added the run ui tests Run the expensive UI tests label Nov 5, 2025
@stopfstedt stopfstedt marked this pull request as ready for review November 5, 2025 19:43
@stopfstedt stopfstedt force-pushed the 6608_replace_verification_preview_table_mixin branch 4 times, most recently from bf83ad6 to 3148eea Compare November 7, 2025 03:33
bolshoi refactoring and kruft killing of table-specific stylesheets that
were consuming the mixin.
@stopfstedt stopfstedt force-pushed the 6608_replace_verification_preview_table_mixin branch from 3148eea to 67a2a5c Compare November 7, 2025 05:05
@jrjohnson jrjohnson removed the request for review from michaelchadwick November 7, 2025 06:16
@stopfstedt
Copy link
Member Author

@dartajax this is a fairly isolated change.

for visual regression review - please compare the netlify preview build to https://demo.iliosproject.org/curriculum-inventory-reports/26/verification-preview.

there should be no changes.

@dartajax
Copy link
Member

dartajax commented Nov 7, 2025

OK - I'm on this one now.

Copy link
Member

@dartajax dartajax left a comment

Choose a reason for hiding this comment

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

looks the same to me - I approve

@dartajax
Copy link
Member

dartajax commented Nov 7, 2025

A special shout out to ... "Percy" - found an issue that does not pertain to this PR but will need to be fixed.

@dartajax dartajax merged commit 6f13179 into ilios:master Nov 7, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run ui tests Run the expensive UI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

verification-preview-table()

3 participants