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

[FLAG-879] Implement new metadata endpoint #4791

Open
wants to merge 23 commits into
base: develop
Choose a base branch
from

Conversation

willian-viana
Copy link
Collaborator

@willian-viana willian-viana commented May 2, 2024

Overview

An overarching explanation of this task can be found on this Miro board:

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 May 2, 2024
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 May 2, 2024 14:26 Inactive
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 May 2, 2024 14:46 Inactive
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 May 3, 2024 17:52 Inactive
pages/api/widget-metadata/[...params].js Dismissed Show dismissed Hide dismissed
pages/api/widget-metadata/[...params].js Dismissed Show dismissed Hide dismissed
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 May 6, 2024 19:29 Inactive
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 May 24, 2024 16:36 Inactive
@willian-viana willian-viana requested a review from wri7tno June 10, 2024 20:21
@willian-viana willian-viana marked this pull request as ready for review June 10, 2024 20:21
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 June 10, 2024 20:22 Inactive
Copy link
Collaborator

@wri7tno wri7tno left a comment

Choose a reason for hiding this comment

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

LGTM, I see the commented code that will replace the hardcoded url and dataset 👍

@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 June 11, 2024 17:51 Inactive
@willian-viana willian-viana force-pushed the feat/new-metadata-endpoint-FLAG-879 branch from 7e4a50e to 0ba4cb4 Compare July 2, 2024 14:42
@willian-viana willian-viana temporarily deployed to gfw-staging-pr-4791 July 2, 2024 14:42 Inactive
@willian-viana willian-viana force-pushed the feat/new-metadata-endpoint-FLAG-879 branch from 0ba4cb4 to aad521c Compare September 9, 2024 14:34
pages/api/metadata/[...params].js Fixed Show fixed Hide fixed
pages/api/metadata/[...params].js Fixed Show fixed Hide fixed
pages/api/metadata/[...params].js Fixed Show fixed Hide fixed
@willian-viana willian-viana force-pushed the feat/new-metadata-endpoint-FLAG-879 branch from 9f878e7 to 57e04fd Compare September 25, 2024 17:26
@willian-viana willian-viana force-pushed the feat/new-metadata-endpoint-FLAG-879 branch from 57e04fd to 56218c8 Compare September 25, 2024 20:44
@willian-viana willian-viana force-pushed the feat/new-metadata-endpoint-FLAG-879 branch from 56218c8 to 453380f Compare September 25, 2024 21:25
@willian-viana willian-viana force-pushed the feat/new-metadata-endpoint-FLAG-879 branch from b94874b to b5aa6e0 Compare December 10, 2024 16:05
solomon-negusse and others added 23 commits December 17, 2024 16:21
For testing purposes I set the dataset and the version as hardcoded
We will remove it as soon as the test get finished
some datasets are not on Data API, for that reason we are still requesting RW API for a few cases
…properties

We have three different properties to display the content date: content_date_description, content_date_range, and content_date, in this order of priority.
We won't use this property anymore
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.

3 participants