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

Moving article_id to bookmark from text table #278

Open
tfnribeiro opened this issue Nov 4, 2024 · 0 comments
Open

Moving article_id to bookmark from text table #278

tfnribeiro opened this issue Nov 4, 2024 · 0 comments

Comments

@tfnribeiro
Copy link
Collaborator

Currently, a bookmark will be associated with a text_id, which points to the Text table, that contains a context and an article_id. Currently, this association is not enforced by a FK constraint, and lead to some errors when a bookmark no longer can find the article_id associated to it. (This was quite rare).

However, further reflection lead us to discuss that instead of associating the text with the article, we should instead associate the bookmark with an article_id, which will then have the span (Text) it appears on. This would allow us in the future to maybe have different types of contexts where the word can be found (e.g. video, photo by the user) which would be a context_id, with a particular span.

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

When branches are created from issues, their pull requests are automatically linked.

1 participant