Skip to content

Conversation

@Vitalius1
Copy link
Contributor

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ npm run change

Description of changes

Fix bad imports causing builds to fail. This NavPage was merged before the new import rules in experiments package were merged so it was causing module not found errors.
From this point any new examples and pages will enforce the correct imports.

@Vitalius1 Vitalius1 requested review from atneik and dzearing April 17, 2018 05:50
@Vitalius1
Copy link
Contributor Author

@dzearing Can we merge this? I see that the bundlesize check is stuck for everyone.

Copy link
Collaborator

@manishgarg1 manishgarg1 left a comment

Choose a reason for hiding this comment

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

:shipit:

@manishgarg1 manishgarg1 self-assigned this Apr 17, 2018
@Vitalius1
Copy link
Contributor Author

@manishgarg1 Are we going to wait for bundlesize?

@manishgarg1 manishgarg1 merged commit 7dabb13 into microsoft:master Apr 17, 2018
@Vitalius1 Vitalius1 deleted the v-vibr/NavImportFix branch May 8, 2018 22:26
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants