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

Perceived performance improvement with Mapbox Static Images API #331

Open
wants to merge 28 commits into
base: release/v1.1.0
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 16 commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
ab88584
Mapbox Component (#267)
alextaing Oct 5, 2022
8c677c6
mapbox component jest tests (#311)
yen-tt Oct 10, 2022
69327c1
add react-dom as peer dep (#313)
yen-tt Oct 11, 2022
731c9bd
docs(MapboxMap): add storybook stories (#312)
tatimblin Oct 18, 2022
0d78e7e
use createRoot when possible for rendering pin components (#319)
oshi97 Oct 28, 2022
f585a30
Revert "use createRoot when possible for rendering pin components (#3…
oshi97 Nov 4, 2022
2c8a500
display mapbox static image until the interactive map is loaded
Nov 9, 2022
a926516
function for url, change interface
Nov 10, 2022
7b89a4f
add story for static image before load
Nov 10, 2022
0bb3220
refactor from hook to component
Nov 10, 2022
261facb
mock response to be error instead of delay
Nov 10, 2022
8faf428
remove msw setup, story use MapboxStaticImage component
Nov 10, 2022
28727e4
wait for map to load for non static map stories
Nov 10, 2022
bb25f48
ts
Nov 10, 2022
91089ce
update percy config in story
Nov 10, 2022
3d5a491
test
Nov 10, 2022
2c70739
hide getCoordinate and onDrag control
Nov 14, 2022
8ad916a
for snapshot of dynamic map with play function
Nov 14, 2022
7ab0c0c
Merge branch 'develop' into dev/mapbox-static-image
Nov 15, 2022
44f0e4a
update package lock for test-site
Nov 15, 2022
def501e
remove play function -- doesnt deplay snapshot
Nov 15, 2022
3538bfb
test dev wcag gh workflow with mapbox api key input
Nov 16, 2022
57f505f
test
Nov 16, 2022
139ee22
remove logs
Nov 16, 2022
6b48f80
update workflow to point back to v1 branch, run WCAG in feature branc…
Nov 16, 2022
06c7725
Fix wcag github action mapbox issues (#337)
yen-tt Nov 17, 2022
1ead573
Fix visual coverage test in Github Actions (#339)
yen-tt Nov 18, 2022
ffe64e5
periods
Nov 18, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/percy-snapshots.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,4 @@ jobs:
percy_script: npx percy storybook ./storybook-static
secrets:
PERCY_TOKEN: ${{ secrets.PERCY_TOKEN }}
MAPBOX_API_KEY: ${{ secrets.MAPBOX_API_KEY }}
2 changes: 1 addition & 1 deletion .github/workflows/run-linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Run Linter

on:
pull_request:
branches: [main, develop]
branches: [main, develop, feature/*]

jobs:
linting:
Expand Down
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,5 @@ lib/
# testing
/coverage
/storybook-static

**/.env
6 changes: 5 additions & 1 deletion .storybook/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,5 +33,9 @@ module.exports = {
config.resolve.alias['./SearchCore'] = require.resolve('../tests/__fixtures__/core/SearchCore.ts');
config.resolve.alias['../utils/location-operations'] = require.resolve('../tests/__fixtures__/utils/location-operations.ts');
return config;
}
},
env: (config) => ({
...config,
REACT_APP_MAPBOX_API_KEY: process.env.MAPBOX_API_KEY || process.env.REACT_APP_MAPBOX_API_KEY,
}),
};
4 changes: 3 additions & 1 deletion .storybook/preview.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import './index.css';
import 'mapbox-gl/dist/mapbox-gl.css';
import { SearchCoreDecorator } from '../tests/__fixtures__/core/SearchCore';
import { runOnly } from './wcagConfig';

Expand All @@ -22,6 +23,7 @@ export const parameters = {
'SearchBar',
'UniversalResults',
'VerticalResults',
'MapboxMap',
'DirectAnswer',
'FilterSearch',
'StaticFilters',
Expand All @@ -42,4 +44,4 @@ export const parameters = {
};

// Add the decorator to all stories
export const decorators = [SearchCoreDecorator];
export const decorators = [SearchCoreDecorator];
Loading