Skip to content

Remove todo statement (see issue #143)#353

Merged
annehaley merged 1 commit intomasterfrom
remove-todo
Mar 19, 2026
Merged

Remove todo statement (see issue #143)#353
annehaley merged 1 commit intomasterfrom
remove-todo

Conversation

@annehaley
Copy link
Copy Markdown
Collaborator

Resolves #143. See issue for discussion.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying geodatalytics with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2e32fca
Status: ✅  Deploy successful!
Preview URL: https://ade5ede1.geodatalytics.pages.dev
Branch Preview URL: https://remove-todo.geodatalytics.pages.dev

View logs

Copy link
Copy Markdown
Collaborator

@brianhelba brianhelba left a comment

Choose a reason for hiding this comment

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

Makes sense to me to track this in an issue rather than a code comment.

I'm not sure if you'd rather have a review of state of this issue from @jjnesbitt though.

@jjnesbitt
Copy link
Copy Markdown
Member

No objection from me.

@annehaley annehaley merged commit c1c133c into master Mar 19, 2026
4 checks passed
@annehaley annehaley deleted the remove-todo branch March 19, 2026 14:37
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.

Add new layer frames in addLayer, not updateLayersShown

3 participants