Skip to content

Conversation

@theerebuss
Copy link
Contributor

Current Behavior

Story asset URL is still pointing to the old path

New Behavior

Story asset URL now pointing to /react-components/react-card

Related Issue(s)

#22692

@ghost
Copy link

ghost commented Apr 29, 2022

CLA assistant check
All CLA requirements met.

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d6aa76b:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam
Copy link
Collaborator

📊 Bundle size report

🤖 This report was generated against 88fa8f279506c3cfbcc2facff95ecb5b420a217a

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 848 851 5000
Button mount 527 470 5000
FluentProvider mount 1805 1883 5000
FluentProviderWithTheme mount 272 301 10
FluentProviderWithTheme virtual-rerender 254 243 10
FluentProviderWithTheme virtual-rerender-with-unmount 289 299 10
MakeStyles mount 1417 1385 50000

@size-auditor
Copy link

size-auditor bot commented Apr 29, 2022

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 88fa8f279506c3cfbcc2facff95ecb5b420a217a (build)

@theerebuss theerebuss changed the title fix: change storybook asset url fix(react-card): change storybook asset url Apr 29, 2022
@theerebuss theerebuss merged commit 14bafc5 into microsoft:master May 2, 2022
@theerebuss theerebuss deleted the card-fix-story-assets branch May 2, 2022 13:47
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
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