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.

Sign up for GitHub

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

Upgrade SDK version #413

Merged
merged 3 commits into from
Nov 26, 2024
Merged

Upgrade SDK version #413

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2d8f5bb
feat: Upgrade SDK version.
andreeaceachir142 Nov 25, 2024
6463cf8
chore: Run npm install.
andreeaceachir142 Nov 25, 2024
b149728
chore: Update to latest version.
andreeaceachir142 Nov 26, 2024
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
2 changes: 1 addition & 1 deletion package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions packages/map-template/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [1.59.0] - 2024-11-26

### Added

- Upgrade SDK version to v4.37.1.

## [1.58.0] - 2024-10-28

### Added
Expand Down
4 changes: 2 additions & 2 deletions packages/map-template/src/components/MapTemplate/MapTemplate.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,8 +180,8 @@

const miSdkApiTag = document.createElement('script');
miSdkApiTag.setAttribute('type', 'text/javascript');
miSdkApiTag.setAttribute('src', 'https://app.mapsindoors.com/mapsindoors/js/sdk/4.36.3/mapsindoors-4.36.3.js.gz');
miSdkApiTag.setAttribute('integrity', 'sha384-UPPUHq6fu2VwcT5Rv+1+FzOlAg6mLd8iAUpr6taDU5XsKNluEpmIenfqKoEBQZPc');
miSdkApiTag.setAttribute('src', 'https://app.mapsindoors.com/mapsindoors/js/sdk/4.37.1/mapsindoors-4.37.1.js.gz');
miSdkApiTag.setAttribute('integrity', 'sha384-wKFOAAB0ZnooQtWKlsw5eY63LFekFS3wI2WRzwTaYUIewjw4Y6mJGo792CqRzbIh');
miSdkApiTag.setAttribute('crossorigin', 'anonymous');
document.body.appendChild(miSdkApiTag);
miSdkApiTag.onload = () => {
Expand All @@ -197,7 +197,7 @@
if (isInactive) {
resetStateAndUI();
}
}, [isInactive]);

Check warning on line 200 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has a missing dependency: 'resetStateAndUI'. Either include it or remove the dependency array

/**
* Wait for the MapsIndoors JS SDK to be initialized, then set the mapsindoorsSDKAvailable state to true.
Expand Down Expand Up @@ -256,7 +256,7 @@

setCurrentLanguage(languageToUse);
}
}, [language, mapsindoorsSDKAvailable]);

Check warning on line 259 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has missing dependencies: 'appConfig.venueImages', 'categories.length', 'currentLanguage', 'currentLocation', 'setCurrentLanguage', 'setCurrentLocation', 'setVenuesInSolution', 'updateCategories', and 'venuesInSolution.length'. Either include them or remove the dependency array

/**
* React on changes in the MapsIndoors API key by fetching the required data.
Expand Down Expand Up @@ -292,7 +292,7 @@
});
setMapReady(false);
}
}, [apiKey, mapsindoorsSDKAvailable]);

Check warning on line 295 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has missing dependencies: 'setApiKey', 'setAppConfig', 'setMapReady', 'setSolution', and 'setVenuesInSolution'. Either include them or remove the dependency array

/*
* Set map provider access token / API key based on props and app config.
Expand All @@ -302,7 +302,7 @@
setMapboxAccessToken(mapboxAccessToken || appConfig.appSettings?.mapboxAccessToken);
setGmApiKey(gmApiKey || appConfig.appSettings?.gmKey);
}
}, [gmApiKey, mapboxAccessToken, mapsindoorsSDKAvailable, appConfig]);

Check warning on line 305 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has missing dependencies: 'setGmApiKey' and 'setMapboxAccessToken'. Either include them or remove the dependency array

/*
* React on changes in the app user roles prop.
Expand Down Expand Up @@ -330,7 +330,7 @@
setFilteredLocationsByExternalID([]);
}
}
}, [externalIDs, mapsindoorsSDKAvailable]);

Check warning on line 333 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has a missing dependency: 'setFilteredLocationsByExternalID'. Either include it or remove the dependency array

/*
* React on changes to the locationId prop.
Expand All @@ -348,7 +348,7 @@
});
}
}
}, [locationId, mapsindoorsSDKAvailable, resetCount]);

Check warning on line 351 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has missing dependencies: 'setCurrentLocation', 'setCurrentVenueName', and 'setLocationId'. Either include them or remove the dependency array

/*
* React to changes in the gmMapId prop.
Expand All @@ -357,7 +357,7 @@
if (mapsindoorsSDKAvailable) {
setGmMapId(gmMapId);
}
}, [gmMapId, mapsindoorsSDKAvailable]);

Check warning on line 360 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has a missing dependency: 'setGmMapId'. Either include it or remove the dependency array

/*
* Add Location to history payload to make it possible to re-enter location details with that Location.
Expand All @@ -378,21 +378,21 @@
// Apply the previously filtered locations to the map when navigating outside the directions.
setFilteredLocations(initialFilteredLocations);
}
}, [currentAppView]);

Check warning on line 381 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has missing dependencies: 'appStates.DIRECTIONS', 'appStates.LOCATION_DETAILS', 'currentAppViewPayload', 'currentLocation', 'filteredLocations', 'initialFilteredLocations', 'setCurrentLocation', and 'setFilteredLocations'. Either include them or remove the dependency array

/*
* React on changes in the venue prop.
*/
useEffect(() => {
setCurrentVenueName(venue);
}, [venue]);

Check warning on line 388 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has a missing dependency: 'setCurrentVenueName'. Either include it or remove the dependency array

/*
* React on changes in the tile style prop.
*/
useEffect(() => {
setTileStyle(tileStyle);
}, [tileStyle]);

Check warning on line 395 in packages/map-template/src/components/MapTemplate/MapTemplate.jsx

View workflow job for this annotation

GitHub Actions / Release Map Template to npm 

React Hook useEffect has a missing dependency: 'setTileStyle'. Either include it or remove the dependency array

/*
* React on changes in the primary color prop.
Expand Down
Loading