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

docs(MapboxMap): add storybook stories #312

Merged
merged 48 commits into from
Oct 18, 2022

Conversation

tatimblin
Copy link
Contributor

@tatimblin tatimblin commented Oct 10, 2022

single marker story
multiple marker story
custom marker story - follow docs.mapbox.com guide to add a simple popup pin

J=SLAP-2223
TEST=manual

serve and view stories

J=SLAP-2223
TEST=manual

visually reviewed while serving storybook
J=SLAP-2223
TEST=manual

serve and view stories
@tatimblin tatimblin requested a review from a team as a code owner October 10, 2022 20:02
@coveralls
Copy link

coveralls commented Oct 10, 2022

Coverage Status

Coverage decreased (-1.2%) to 84.687% when pulling 2e46f46 on dev/mapbox-story into 8c677c6 on feature/mapbox-component.

@github-actions
Copy link
Contributor

Current unit coverage is 89.12721893491124%
Current visual coverage is 75.51020408163265%
Current combined coverage is 89.64497041420118%

.storybook/preview.js Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
J=SLAP-2223
TEST=none
.storybook/tailwind.config.cjs Outdated Show resolved Hide resolved
sample.env Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
test-site/src/pages/LocationsPage.tsx Outdated Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
test-site/src/components/MapPin.tsx Outdated Show resolved Hide resolved
tests/components/MapboxMap.stories.tsx Outdated Show resolved Hide resolved
J=SLAP-2223
TEST=manual

ran storybook and eslint
@tatimblin tatimblin requested a review from nmanu1 October 12, 2022 14:06
@tatimblin tatimblin requested a review from nmanu1 October 13, 2022 20:01
test-site/src/components/MapPin.tsx Show resolved Hide resolved
.storybook/main.js Show resolved Hide resolved
@tatimblin
Copy link
Contributor Author

Still not able to get the map to render in the percy snapshot even though the testing-library checks pass which check that the map was loaded. Think I'll need some help on this

@nmanu1 nmanu1 merged commit 731c9bd into feature/mapbox-component Oct 18, 2022
@nmanu1 nmanu1 deleted the dev/mapbox-story branch October 18, 2022 14:58
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.

6 participants