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

[EPIC] GADM #4892

Draft
wants to merge 10 commits into
base: develop
Choose a base branch
from
Draft

[EPIC] GADM #4892

wants to merge 10 commits into from
+112 −75

Conversation

willian-viana
Copy link
Collaborator

Overview

Brief description of what this PR does, and why it is needed.

Demo

If applicable: screenshots, gifs, etc.

Notes

If applicable: ancilary topics, caveats, alternative strategies that didn't work out, etc.

Testing

  • Include any steps to verify the features this PR introduces.
  • If this fixes a bug, include bug reproduction steps.

@willian-viana willian-viana self-assigned this Nov 25, 2024
@willian-viana willian-viana added the epic This is a branch with multiple features to be delivered all at once label Dec 17, 2024
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4892 January 2, 2025 19:08 Inactive
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4892 January 23, 2025 22:02 Inactive
willian-viana
willian-viana commented Feb 24, 2025
View reviewed changes
pages/api/data/[...params].js
const GFW_API_KEY = process.env.NEXT_PUBLIC_GFW_API_KEY;
const DATA_API_URL =
ENVIRONMENT === 'staging' ? GFW_STAGING_DATA_API : GFW_DATA_API;
const DATA_API_URL = GFW_DATA_API;
Copy link
Collaborator Author

@willian-viana willian-viana Feb 24, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Rollback this change before deploy it to PROD.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gtempus gtempus Awaiting requested review from gtempus

@wri7tno wri7tno Awaiting requested review from wri7tno

At least 1 approving review is required to merge this pull request.

Assignees

@willian-viana willian-viana

Labels
epic This is a branch with multiple features to be delivered all at once
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@willian-viana