Skip to content

Add related contents to content cards #528

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

Open
wants to merge 217 commits into
base: main
Choose a base branch
from

Conversation

lickem22
Copy link
Contributor

Reviewer: @suzinyou
Estimate: 30mins


Ticket

Fixes: JIRA_TICKET_LINK

Description

Goal

The goal of this PR is to add related contents to content cards.

Changes

  • Added related_content_ids field to ContentDB
  • Added a check to make sure all content_ids added exists and are not archived or part of a different workspace.
  • Added an AutoComplete to add related contents on frontend
  • Added field to display related content titles on content modal. Made it clickable.

Future Tasks (optional)

  • Better way of validating contents. Right now when a related content is deleted it is still part of related_content_id until updated
  • When validating related_contents, I filter out non-existing contents but still add existing ones
  • Add latest updates

How has this been tested?

  • ran tests
  • Added related_contents using frontend and api

To-do before merge (optional)

Checklist

Fill with x for completed.

  • My code follows the style guidelines of this project
  • I have reviewed my own code to ensure good quality
  • I have tested the functionality of my code to ensure it works as intended
  • I have resolved merge conflicts

(Delete any items below that are not relevant)

  • I have updated the automated tests
  • I have updated the scripts in scripts/
  • I have updated the requirements
  • I have updated the README file
  • I have updated affected documentation
  • I have added a blogpost in Latest Updates
  • I have updated the CI/CD scripts in .github/workflows/
  • I have updated the Terraform code

MustafaAkolawala and others added 30 commits October 7, 2024 03:21
…Added /chat endpoint. Temporariliy commented out /search endpoint for quick testing.
@suzinyou
Copy link
Collaborator

Why does this have 200 commits ?? lol

Copy link
Collaborator

@suzinyou suzinyou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few UI bugs but the API works very nicely! Thank you so much :D

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image Should we exclude the content that's being edited?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, when I open the list of options and don't select anything (i.e. click outside the list box to escape, it ends up adding the last item that was highlighted.

Screen.Recording.2025-04-16.at.6.19.31.PM.mov

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @suzinyou, I added the fixes. I know you have approved. But feel free to double check

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.

4 participants