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

Add shareable URLs #138

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MetroConductor
Copy link

@MetroConductor MetroConductor commented Oct 17, 2023

Proposed fix for #130. Cause the URL to change on input, and the input to change with the URL. Adds the following URL formats:

  • /#!/{department code}
  • /#!/{postal code}
  • /#!/{postal code}/{section ID}
  • /#!/{postal code}/{section ID}/{plot number}
  • /#!/{plot code}
  • /#!/{from date}-{until date}/{department code}
  • /#!/{from date}-{until date}/{postal code}
  • /#!/{from date}-{until date}/{postal code}/{section ID}
  • /#!/{from date}-{until date}/{postal code}/{section ID}/{plot number}
  • /#!/{from date}-{until date}/{plot code}

Cause the URL to change on input, and the input to change with the URL. Adds the following URL formats:
- /#!/{department code}
- /#!/{postal code}
- /#!/{postal code}/{section ID}
- /#!/{postal code}/{section ID}/{plot number}
- /#!/{plot code}
- /#!/{from date}-{until date}/{department code}
- /#!/{from date}-{until date}/{postal code}
- /#!/{from date}-{until date}/{postal code}/{section ID}
- /#!/{from date}-{until date}/{postal code}/{section ID}/{plot number}
- /#!/{from date}-{until date}/{plot code}
@MetroConductor MetroConductor changed the title Add hashbang URL functionality Add shareable URLs Oct 17, 2023
Change selection queue system to work correctly in all scenarios
The function resetParcelle() was called in changerSelection() while the map wasn't fully loaded, causing an error.
This was fixed by checking the mapRendered boolean before said call, as it doesn't have any effect while the map is still loading any way.
Comment on lines +827 to +829
})
map.addLayer(departementsLayer)
map.addLayer(departementsContoursLayer)
Copy link
Author

Choose a reason for hiding this comment

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

Removed unnecessary indentation to improve readability

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

Successfully merging this pull request may close these issues.

1 participant