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.

Sign up for GitHub

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

Jump to bottom

CARTO: Fix website example #8239

Merged
merged 4 commits into from Nov 7, 2023
Merged

CARTO: Fix website example #8239

merged 4 commits into from Nov 7, 2023

Conversation

felixpalmer
Copy link
Collaborator

For #8148

Change List

  • Update to latest API
  • Disable workers for now

mdeividas
mdeividas approved these changes Oct 31, 2023
View reviewed changes
Copy link

@mdeividas mdeividas left a comment

Choose a reason for hiding this comment

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

LGTM

@felixpalmer felixpalmer mentioned this pull request Oct 31, 2023
Closed
23 tasks
@felixpalmer felixpalmer changed the base branch from felix/carto-request-token to master November 2, 2023 09:21
@felixpalmer felixpalmer changed the base branch from master to felix/carto-request-token November 2, 2023 09:21
Base automatically changed from felix/carto-request-token to master November 7, 2023 12:27
@felixpalmer felixpalmer merged commit a8b22e4 into master Nov 7, 2023
2 checks passed
@felixpalmer felixpalmer deleted the felix/carto-fix-example2 branch November 7, 2023 16:46
@felixpalmer felixpalmer mentioned this pull request Jan 17, 2024
Open
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alasarr alasarr alasarr approved these changes

@mdeividas mdeividas mdeividas approved these changes

@zbigg zbigg Awaiting requested review from zbigg

@padawannn padawannn Awaiting requested review from padawannn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@felixpalmer @alasarr @mdeividas