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

Rename database from "openledger" #666

Open
1 task
AetherUnbound opened this issue Oct 7, 2022 · 0 comments
Open
1 task

Rename database from "openledger" #666

AetherUnbound opened this issue Oct 7, 2022 · 0 comments
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: api Related to the Django API 🧱 stack: catalog Related to the catalog and Airflow DAGs 🔧 tech: django Involves Django 💾 tech: postgres Involves PostgreSQL 🐍 tech: python Involves Python
Projects
Openverse

Comments

@AetherUnbound
Copy link
Contributor

Description

We currently have a number of references to openledger in this codebase, mostly in reference to the Postgres database name. Changing the database name would be a nontrivial effort, but openledger is a historic term that isn't descriptive, conflates with the catalog's current database name, and isn't used anywhere else.

Additional context

Similar to #1546

Implementation

  • 🙋 I would be interested in implementing this feature.
@AetherUnbound AetherUnbound added 🐍 tech: python Involves Python 💾 tech: postgres Involves PostgreSQL 🔧 tech: django Involves Django 🟩 priority: low Low priority and doesn't need to be rushed 🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users labels Oct 7, 2022
@obulat obulat transferred this issue from WordPress/openverse-api Feb 22, 2023
@AetherUnbound AetherUnbound added 🧱 stack: api Related to the Django API 🧱 stack: catalog Related to the catalog and Airflow DAGs and removed 🧱 stack: backend labels May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🧰 goal: internal improvement Improvement that benefits maintainers, not users 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: api Related to the Django API 🧱 stack: catalog Related to the catalog and Airflow DAGs 🔧 tech: django Involves Django 💾 tech: postgres Involves PostgreSQL 🐍 tech: python Involves Python
2 participants
@AetherUnbound @obulat