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

Remove line regarding moving the catalog into the repository from the README #3202

Closed
AetherUnbound opened this issue Oct 16, 2023 · 3 comments · Fixed by #3529
Closed

Remove line regarding moving the catalog into the repository from the README #3202

AetherUnbound opened this issue Oct 16, 2023 · 3 comments · Fixed by #3529
Assignees
@lucasferrazlima
Labels
📄 aspect: text Concerns the textual material in the repository 🛠 goal: fix Bug fix good first issue New-contributor friendly help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: documentation Related to Sphinx documentation

Comments

@AetherUnbound
Copy link
Contributor

Description

These lines can be removed (or maybe rephrased to denote that we do not have our infrastructure declaration in the monorepo):

openverse/README.md

Lines 24 to 25 in aa3a7f5

This repository contains most of the codebase, except for the Openverse catalog.
Eventually the catalog will be merged into this repository as well.

The catalog has been added to the monorepo so this is no longer accurate.
@AetherUnbound AetherUnbound added good first issue New-contributor friendly help wanted Open to participation from the community 📄 aspect: text Concerns the textual material in the repository 🛠 goal: fix Bug fix 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: documentation Related to Sphinx documentation labels Oct 16, 2023
@AlajeBash
Copy link
Contributor

Hi i will like to contribute to this issue by rephrasing the statements on line 24 and 25 to "This repository contains most of the codebase, including the Openverse catalog, which has now been integrated into the monorepo." . Please assign me Thanks

@dhruvkb
Copy link
Member

dhruvkb commented Oct 22, 2023

@AlajeBash it would be better to drop that paragraph entirely as it's not wise to have time-specific information in the README.md. Assigning it to you.

@AlajeBash AlajeBash mentioned this issue Oct 23, 2023
Closed
8 tasks
@lucasferrazlima
Copy link
Contributor

lucasferrazlima commented Dec 15, 2023 •
edited

@AetherUnbound @dhruvkb It's been a while since this issue was assigned.
I'd love to help move it forward, as it seems like a straightforward task.

@dhruvkb
Copy link
Member

dhruvkb commented Dec 15, 2023 •
edited

Thanks @lucasferrazlima, I'll assign it to you.

@dhruvkb dhruvkb assigned lucasferrazlima and unassigned AlajeBash Dec 15, 2023
lucasferrazlima added a commit to lucasferrazlima/openverse that referenced this issue Dec 15, 2023
@lucasferrazlima
Remove outdated catalog info from README (WordPress#3202)
e2619f0
@lucasferrazlima lucasferrazlima mentioned this issue Dec 15, 2023
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lucasferrazlima lucasferrazlima

Labels
📄 aspect: text Concerns the textual material in the repository 🛠 goal: fix Bug fix good first issue New-contributor friendly help wanted Open to participation from the community 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: documentation Related to Sphinx documentation
Projects
Openverse Backlog
Archived in project
Milestone
No milestone
4 participants
@AetherUnbound @dhruvkb @AlajeBash @lucasferrazlima