Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Sanitize zoom sequence value #837

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Sanitize zoom sequence value #837

wants to merge 1 commit into from

Conversation

otacke
Copy link

@otacke otacke commented Jun 19, 2023

When merged in, will sanitize the zoom_sequence value passed to Timeline.

In particular, it will ...

  • turn properly formatted strings from URL fragments into arrays,
  • remove invalid values, and
  • sort the sequence.

Fixes #829

@otacke otacke mentioned this pull request Sep 27, 2023
Open
@oleksandr-danylchenko
Copy link
Contributor

Looks neat!

@otacke
Copy link
Author

otacke commented Mar 15, 2024

@oleksandr-danylchenko Thanks! Hope that @JoeGermuska may find some time to review it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

zoom_sequence option not supported via URL parameter
2 participants
@otacke @oleksandr-danylchenko