Skip to content

Support participation-condition Storybook fixture workflows#62

Closed
coder13 wants to merge 4 commits intomainfrom
participation-condition
Closed

Support participation-condition Storybook fixture workflows#62
coder13 wants to merge 4 commits intomainfrom
participation-condition

Conversation

@coder13
Copy link
Copy Markdown
Owner

@coder13 coder13 commented Apr 17, 2026

Summary

  • add broad competition page Storybook coverage and container-based competition page wrappers
  • add a Storybook 8-native fixture switcher for Group stories using story parameters fixtures
  • pin strip-ansi for Jest compatibility so the push hook passes again

Verification

  • yarn check:type
  • yarn build-storybook
  • yarn test

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 17, 2026

Deploy Preview for competition-groups ready!

Name Link
🔨 Latest commit cb09e3e
🔍 Latest deploy log https://app.netlify.com/projects/competition-groups/deploys/69e6b2794d52ef0008e895ef
😎 Deploy Preview https://deploy-preview-62--competition-groups.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 68
Accessibility: 84
Best Practices: 100
SEO: 96
PWA: 90
View the detailed breakdown and full score reports

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

@coder13 coder13 force-pushed the participation-condition branch from b808cca to 433c1e4 Compare April 20, 2026 17:21
@coder13 coder13 force-pushed the participation-condition branch from 433c1e4 to ef9d104 Compare April 20, 2026 18:50
: 'common.wca.advancement.resultAchieved',
{
defaultValue: isLinkedRounds
? 'Competitors with a {{scope}} better than {{result}} combined across {{rounds}} advance to next round. Minimum of 25% of competitors must be eliminated.'
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

This bit of text needs to be moved to the translation files.

const cutoff = round.cutoff;
const timeLimit = round.timeLimit;
const timelimitTime = timeLimit && renderCentiseconds(timeLimit?.centiseconds);
const eventRounds = useMemo(() => {
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

This looks like generic code we could move to a helper function.

@coder13 coder13 closed this Apr 20, 2026
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.

1 participant