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

Add object to list #272

Merged
merged 12 commits into from
Oct 20, 2023
Merged

Add object to list #272

merged 12 commits into from
Oct 20, 2023

Conversation

barbarah
Copy link
Contributor

@barbarah barbarah commented Oct 6, 2023

This pull request will add a dropdown to the object detail page. The user can choose a list to add the object to. After adding the object, the object is shown on the community page.

The design of the object list dropdown I made myself, and maybe needs some changes later.

I made two schema changes:

1: objectItem.listId could be null. An objectItem without a list makes no sense, so I made this property not null.
2: The objectItem.objectId (hashed URI) was the primary key. But objectId is not unique because objects can be added to multiple lists. So, I have added objectItem.id with an auto-increment and made this the primary key.

@vercel
Copy link

vercel bot commented Oct 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
colonial-collections-dataset-browser ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 20, 2023 7:59am
colonial-collections-researcher ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 20, 2023 7:59am

@github-actions
Copy link

github-actions bot commented Oct 6, 2023

Database schema changes have been applied

The development database migrated with the migrations found in this branch.

Warning: All preview and development environments use the same database. This migration can affect all preview and development environments.

After merging this pull request, the development database changes will merge into the production database.

@barbarah
Copy link
Contributor Author

barbarah commented Oct 6, 2023

@sdevalk, I just removed you as a reviewer. There is a bug in my last changes.

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.

2 participants