Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(gatsby): remove resource query from warnings #36439

Merged
merged 1 commit into from
Aug 23, 2022

Conversation

marvinjude
Copy link
Contributor

@marvinjude marvinjude commented Aug 22, 2022

Description

warn ./src/pages/index.js?export=default
Attempted import error: 'red' is not exported from './index.module.scss' (imported as 'styles').
warn ./src/pages/index.js?export=head
Attempted import error: 'red' is not exported from './index.module.scss' (imported as 'styles').

With the introduction of Head API, the modulePath can have a resourceQuery so this removes ?export=head from the error above

[sc-53668]

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 22, 2022
@marvinjude marvinjude added topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters) and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Aug 22, 2022
@marvinjude marvinjude marked this pull request as ready for review August 22, 2022 15:15
@marvinjude marvinjude merged commit 1bf2358 into master Aug 23, 2022
@marvinjude marvinjude deleted the remove-resource-query-from-attemped-import-error branch August 23, 2022 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants