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

Clearly document all media properties #412

Closed
3 tasks done
Tracked by #2251
obulat opened this issue Feb 16, 2023 · 11 comments
Closed
3 tasks done
Tracked by #2251

Clearly document all media properties #412

obulat opened this issue Feb 16, 2023 · 11 comments
Assignees
Labels
🧭 project: thread An issue used to track a project and its progress

Comments

@obulat
Copy link
Contributor

obulat commented Feb 16, 2023

Start Date Project Lead Actual Ship Date
2023-03-01 @dhruvkb TBD

Description

All the details we collect and store for each media type should be clearly documented, and should be consistent across the stack (Catalog, API and Frontend).

Due to historic reasons during the development of the project, not all properties that are deemed required are available in the data. As a pre-requisite to making the data more consistent, we need to document all the media properties and whether they are available (should be available in the future) in the database or are added by the API/ingestion server/elasticsearch/frontend.

Also, we need to document whether the properties are nullable, and what their constraints are. Although we have the database constraints, some of them are not reliable.

Documents

Implementation Plans

Issues

Prior Art

@obulat
Copy link
Contributor Author

obulat commented Apr 26, 2023

Update 2023-04-26

The project proposal has been finalized, and the PR with the Catalog implementation plan (#1904) opened.
The migration to drf-spectacular that creates OpenAPI 3 spec in the API is also the foundation work for the API and frontend parts of this project.

Blockers

None.

@openverse-bot

This comment was marked as outdated.

2 similar comments
@openverse-bot
Copy link
Collaborator

Hi @obulat, this project has not received an update comment in 14 days. Please leave an update comment as soon as you can. See the documentation on project updates for more information.

@openverse-bot
Copy link
Collaborator

Hi @obulat, this project has not received an update comment in 14 days. Please leave an update comment as soon as you can. See the documentation on project updates for more information.

@zackkrida zackkrida moved this from In RFC to On Hold in Openverse Project Tracker Jul 5, 2023
@AetherUnbound AetherUnbound moved this from ⏸ On Hold to 🚧 In Progress in Openverse Project Tracker Jan 3, 2024
@AetherUnbound AetherUnbound assigned dhruvkb and unassigned obulat Jan 3, 2024
@dhruvkb
Copy link
Member

dhruvkb commented Apr 3, 2024

Update 2024-04-03:

Both implementation plans for the project have been approved and merged. Both implementation plans also have implementation PRs.

IP request IP PR IP impl request IP impl PR
#1995 #1904 #2186 #3620
#1996 #3965 #4023 #3966

When both implementation scripts are merged, we will move this project to "Shipped" where it will stay while manual field documentation is being written. That is covered by these issues:

@dhruvkb dhruvkb moved this from 🚧 In Progress to 🚢 Shipped in Openverse Project Tracker Apr 13, 2024
@openverse-bot
Copy link
Collaborator

Hi @dhruvkb, this project has not received an update comment in 14 days. Please leave an update comment as soon as you can. See the documentation on project updates for more information.

1 similar comment
@openverse-bot
Copy link
Collaborator

Hi @dhruvkb, this project has not received an update comment in 14 days. Please leave an update comment as soon as you can. See the documentation on project updates for more information.

@dhruvkb
Copy link
Member

dhruvkb commented May 2, 2024

This project has been shipped, efforts are underway to describe the properties manually across catalog, API and frontend, after which it can be considered successful.

@zackkrida
Copy link
Member

I've begun work on #4025 and should have a PR out in the next 1-2 days.

@openverse-bot
Copy link
Collaborator

Hi @dhruvkb, this project has not received an update comment in 14 days. Please leave an update comment as soon as you can. See the documentation on project updates for more information.

@dhruvkb
Copy link
Member

dhruvkb commented May 30, 2024

#4366 is the final PR which will close #2187 and also this project.

@zackkrida zackkrida moved this from 🚢 Shipped to ✅ Success in Openverse Project Tracker Jun 5, 2024
@github-project-automation github-project-automation bot moved this from ✅ Success to 🚢 Shipped in Openverse Project Tracker Jun 5, 2024
@zackkrida zackkrida moved this from 🚢 Shipped to ✅ Success in Openverse Project Tracker Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧭 project: thread An issue used to track a project and its progress
Projects
Status: ✅ Success
Development

No branches or pull requests

5 participants