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: importing types from a local file instead of the package in SfBadge #2939

Merged
merged 3 commits into from
Aug 17, 2023

Conversation

filrak
Copy link
Collaborator

@filrak filrak commented Jul 27, 2023

Related issue

Closes #

Scope of work

I just changed the import statement to point to the package

Screenshots of visual changes

Checklist

  • Self code-reviewed
  • Changes documented
  • Semantic HTML
  • SSR-friendly
  • Caching friendly
  • a11y for WCAG 2.0 AA
  • examples created
  • blocks created
  • cypress tests created

@changeset-bot
Copy link

changeset-bot bot commented Jul 27, 2023

🦋 Changeset detected

Latest commit: 80c0da2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@storefront-ui/react Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Szymon-dziewonski
Copy link
Contributor

Szymon-dziewonski commented Jul 31, 2023

Hey @filrak, thank you for contribution, however there are couple issues with code. I left one comment and also please replace exports in SfBadge/index.ts
into

export * from './types';

export { default as SfBadge } from './SfBadge';

This is due to how aliases locally works in vite, so we would not need to build during developement.

Also since this PR is patch into @storefront/react package it needs to have changeset. In order to create one please run yarn changeset and fallow prompt cli instructions. Thanks !

If its more suitable for you I can take it over and finish this PR.

filrak and others added 2 commits August 16, 2023 14:16
@Szymon-dziewonski Szymon-dziewonski changed the title fix: SfBadge importing types from a local file instead of the package fix: importing types from a local file instead of the package in SfBadge Aug 17, 2023
@sonarcloud
Copy link

sonarcloud bot commented Aug 17, 2023

[storefront-ui-vue] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@sonarcloud
Copy link

sonarcloud bot commented Aug 17, 2023

[storefront-ui-react] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@Szymon-dziewonski Szymon-dziewonski merged commit d57c3cf into v2-develop Aug 17, 2023
19 of 28 checks passed
@Szymon-dziewonski Szymon-dziewonski deleted the filrak-patch-1 branch August 17, 2023 14:48
Szymon-dziewonski pushed a commit that referenced this pull request Oct 2, 2023
…dge (#2939)

---------
Co-authored-by: Szymon Dziewoński <szymon.dziewonski@gmail.com>
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.

None yet

3 participants