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

refactor: switch many examples to TypeScript #469

Merged
merged 5 commits into from
Oct 21, 2024

Conversation

KiwiKilian
Copy link
Collaborator

@KiwiKilian KiwiKilian commented Oct 15, 2024

Description

I've migrated and improved quite a few examples:

  • Fixes partially Fix/clean up examples #12
  • User Location examples got some new functionality
  • Fixed AnimateCircleAlongLine with basic LineString
  • Upgraded react-native-elements
    • Not well maintained, looking to get rid of it
  • Got rid of prop-types
  • Caveats:
    • I had to use ts-ignore twice for animated styles (see AnimateCircleAlongLine)
    • Example.tsx has a few any, thinking to replace with expo-router file base routing maybe?

Happy to rebase, once #467 is merged.

Checklist

  • I have tested this on a device/simulator for each compatible OS
  • I formatted JS and TS files with running yarn lint:fix in the root folder
  • I have run tests via yarn test in the root folder
  • I updated the documentation with running yarn generate in the root folder
  • I mentioned this change in CHANGELOG.md
  • I updated the typings files (index.d.ts)
  • I added/updated a sample (/example)

# Conflicts:
#	packages/examples/package.json
#	packages/examples/src/examples/Animations/DriveTheLine.js
#	packages/examples/src/examples/Annotations/MarkerView.js
#	packages/examples/src/examples/Annotations/ShowPointAnnotation.js
#	packages/examples/src/examples/FillRasterLayer/IndoorBuilding.js
#	packages/examples/src/examples/FillRasterLayer/WatercolorRasterTiles.js
#	packages/examples/src/examples/Map/ShowMap.tsx
#	packages/examples/src/examples/SymbolCircleLayer/EarthQuakes.js
#	packages/examples/src/examples/UserLocation/SetDisplacement.js
#	packages/examples/src/examples/UserLocation/SetTintColor.js
#	packages/examples/src/examples/UserLocation/SetUserLocationRenderMode.js
#	packages/examples/src/examples/UserLocation/SetUserLocationVerticalAlignment.js
#	packages/examples/src/examples/UserLocation/UserLocationChange.js
#	packages/examples/src/examples/common/Bubble.js
#	packages/examples/src/examples/common/MapHeader.js
#	packages/examples/src/examples/common/PulseCircleLayer.js
#	packages/examples/src/examples/common/TabBarPage.js
#	packages/examples/src/examples/index.js
#	packages/examples/src/scenes/Examples.js
#	yarn.lock
@tyrauber tyrauber merged commit 4457490 into maplibre:main Oct 21, 2024
4 checks passed
@tyrauber
Copy link
Collaborator

Thank you @KiwiKilian! Huge improvement.

@KiwiKilian KiwiKilian deleted the feat/examples-ts branch October 21, 2024 18:48
Copy link

github-actions bot commented Dec 2, 2024

🎉 This PR is included in version 10.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants