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

v1.2.0 #65

Merged
merged 65 commits into from
Nov 1, 2023
Merged

v1.2.0 #65

merged 65 commits into from
Nov 1, 2023

Conversation

jonathanlurie
Copy link
Collaborator

@jonathanlurie jonathanlurie commented Oct 31, 2023

RD-173

  • adding changelog
  • adding gh action to format and lint for PR

Craig O'Connor and others added 30 commits August 16, 2023 15:34
Updating to Maplibre 3.3.0
jonathanlurie and others added 22 commits August 24, 2023 15:45
Co-authored-by: Craig O'Connor <[email protected]>
Co-authored-by: Craig O'Connor <[email protected]>
Co-authored-by: Craig O'Connor <[email protected]>
Co-authored-by: Craig O'Connor <[email protected]>
ENH now supporting all the languages supported by OMT
* ADD visitor language

* WIP language logic revamp

* ADD language from style metadata

* ENH language definition

* REM got rid of commented code

* ENH adjusted the size of secondary lang with visitor language mode

* Update src/Map.ts

Co-authored-by: Craig O'Connor <[email protected]>

* Update src/Map.ts

Co-authored-by: Craig O'Connor <[email protected]>

* ENH from review

* ENH added things from review

---------

Co-authored-by: Craig O'Connor <[email protected]>
* ADD visitor language

* ADD helper for addong polygon from geojson

* ADD map.addPolygon

* REM some data

* Update src/Map.ts

Co-authored-by: Craig O'Connor <[email protected]>

* ENH unused import

* Update src/Map.ts

Co-authored-by: Craig O'Connor <[email protected]>

* Update src/Map.ts

Co-authored-by: Craig O'Connor <[email protected]>

* ENH from review

---------

Co-authored-by: Craig O'Connor <[email protected]>
* first attempt

* reduce typescript warnings

* reduce typescript warnings

* add final todo

* zoom ceiling; bounds bug fix; pitch off by default; unique style; simplify; etc.

* fix key

* simplify typescript

* review part 1; cleaner types; better comments

* remove zoomCeiling; its essentially minZoom

* improve readability of minimap options input

* bug fix; adjustments

* add style downwind functions; requested changes

* slight adjustment

* set lang fix

* improve types

---------

Co-authored-by: Craig O'Connor <[email protected]>
* WIP point layer helper

* WIP better clusters

* WIP changed point helper options

* ADD color ramps with builtins

* WIP simpler set of options

* ADD non linear resampling on color ramps

* ADD color ramp doc

* ADD outline color for points

* ADD point outline settings

* REM colorRamp arg and use only pointColor instead

* DONE Map.addPoint
* WIP point layer helper

* WIP better clusters

* WIP changed point helper options

* ADD color ramps with builtins

* WIP simpler set of options

* ADD non linear resampling on color ramps

* ADD color ramp doc

* ADD outline color for points

* ADD point outline settings

* REM colorRamp arg and use only pointColor instead

* DONE Map.addPoint

* ADD radius and weight being both zoom and property driven

* ADD zoom comensation param to point helper

* FIX minmaxzoom and added MapTiler cloud source with uuid

* DOC polyline helper section in readme

* WIP readme

* DOC done with documenting helpers

* DOC color ramp doc

* DOC fix

* DOC

* ENH cleaning unused types

* ENH updating eslint

* FIX dependecy audit

* ENH export types and a better style path

* ENH refectoring to place the helpers in a dedicated place

* FIX typo
* WIP point layer helper

* WIP better clusters

* WIP changed point helper options

* ADD color ramps with builtins

* WIP simpler set of options

* ADD non linear resampling on color ramps

* ADD color ramp doc

* ADD outline color for points

* ADD point outline settings

* REM colorRamp arg and use only pointColor instead

* DONE Map.addPoint

* ADD radius and weight being both zoom and property driven

* ADD zoom comensation param to point helper

* FIX minmaxzoom and added MapTiler cloud source with uuid

* DOC polyline helper section in readme

* WIP readme

* DOC done with documenting helpers

* DOC color ramp doc

* DOC fix

* DOC

* ENH cleaning unused types

* ENH updating eslint

* FIX dependecy audit

* ENH export types and a better style path

* ENH refectoring to place the helpers in a dedicated place

* ENH changed TypeScript moduleResolution from Node to Bundler
* WIP point layer helper

* WIP better clusters

* WIP changed point helper options

* ADD color ramps with builtins

* WIP simpler set of options

* ADD non linear resampling on color ramps

* ADD color ramp doc

* ADD outline color for points

* ADD point outline settings

* REM colorRamp arg and use only pointColor instead

* DONE Map.addPoint

* ADD radius and weight being both zoom and property driven

* ADD zoom comensation param to point helper

* FIX minmaxzoom and added MapTiler cloud source with uuid

* DOC polyline helper section in readme

* WIP readme

* DOC done with documenting helpers

* DOC color ramp doc

* DOC fix

* DOC

* ENH cleaning unused types

* ENH updating eslint

* FIX dependecy audit

* ENH export types and a better style path

* ENH refectoring to place the helpers in a dedicated place

* ENH changed TypeScript moduleResolution from Node to Bundler

* UP now using Maplibre v3.5.2 (strict)
@jonathanlurie jonathanlurie changed the title Dev v1.2.0 v1.2.0 Oct 31, 2023
Copy link
Member

@petrsloup petrsloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are now some lint errors, but the workflow does not fail.
The format/lint rules no actually make the changes to the local files, so that's probably why the workflow does not fail.
It would probably be better to have to versions of these: format/lint just for checking (useful for CI and just checking) and format:fix/lint:fix that actually make to changes (for running during development).

@petrsloup petrsloup self-requested a review November 1, 2023 11:25
Copy link
Member

@petrsloup petrsloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jonathanlurie jonathanlurie merged commit 87568d2 into main Nov 1, 2023
1 check passed
@jonathanlurie jonathanlurie deleted the dev-v1.2.0 branch November 1, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants