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

feat(geo): update queryUrl with min max resolution #1167

Merged
merged 21 commits into from
Mar 14, 2023

Conversation

aziz-access
Copy link
Contributor

What is the current behavior? (You can also link to an open issue here)

this update relate to the issue #984 Custom Query Url with min/max resolution

What is the new behavior?

add the possibility to change the url depending on the zoom/resolution of the map.

@aziz-access aziz-access added enhancement New features and improvements // Nouvelles fonctionnalités et améliorations in progress In progress // Développement en cours labels Jan 18, 2023
@aziz-access aziz-access self-assigned this Jan 18, 2023
@PhilippeLafreniere18 PhilippeLafreniere18 added ready to merge and removed in progress In progress // Développement en cours labels Feb 9, 2023
Copy link
Member

@pelord pelord left a comment

Choose a reason for hiding this comment

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

I did not test the pr.. i'll do it soon ;)

packages/geo/src/lib/query/shared/query.service.ts Outdated Show resolved Hide resolved
packages/geo/src/lib/query/shared/query.interfaces.ts Outdated Show resolved Hide resolved
@PhilippeLafreniere18
Copy link
Contributor

@matrottier veux-tu qu'on continue d'honorer l'objet initial ? On le changerait pour un array ici (queryUrls au lieu de queryUrl). À toi de voir...

@matrottier
Copy link
Contributor

@matrottier veux-tu qu'on continue d'honorer l'objet initial ? On le changerait pour un array ici (queryUrls au lieu de queryUrl). À toi de voir...

semble tiguidou ! go ! go ! go !

@pelord pelord merged commit 5b2cf96 into next Mar 14, 2023
@pelord pelord deleted the customQueryUrlWithMinMaxParam branch March 14, 2023 16:54
cbourget pushed a commit that referenced this pull request Mar 21, 2023
* feat(geo): update queryUrl with min max resolution

* solve lint errors

* solve lint errors..

* solve lint errors

* test query

* test

* fix(query): fix query url bbox param

* wip

* branch testing new code

* merge with queryUrlFix

* add resolution and scale check

* lint

* check scale and resolution are exists

* verification of resolution and scale

* refactor/fix(queryUrl): fix conditions and examples for scale and resolution

* fix typo

* fix(query): fix htmlgml query geom

* fix(query): htmlgml2 geometries were undefined or wrong

---------

Co-authored-by: Philippe Lafreniere <[email protected]>
Co-authored-by: Pierre-Étienne Lord <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features and improvements // Nouvelles fonctionnalités et améliorations ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants