Skip to content

Conversation

@GspikeHalo
Copy link
Contributor

@GspikeHalo GspikeHalo commented Oct 4, 2024

Propose:
Add the like button while refactoring the clone button.

Changes:

  1. Added two new tables to the database to track likes and clones. Use 16.sql to update the database.
  2. Added a like button to the list item, which will be displayed in the hub interface, but disabled when the user is not logged in.
  3. Added a like button and a clone button to the detail page, both of which will be disabled when the user is not logged in. The clone button will be disabled in the detail preview interface

Demo:
Old:
list item before login:
old list item before login

detail page before login:
old detail page before login

list item after login:
old list item after login

detail page after login:
old detail page after login

New:
list item before login:
new hub list item before login

detail page before login:
new detail page before login

list item after login:
new hub list item after login

detail page after login:
new detail page after login

@GspikeHalo GspikeHalo marked this pull request as ready for review October 9, 2024 20:10
@sixsage sixsage added ddl-change Changes to the TexeraDB DDL starter-task labels Oct 11, 2024
Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

Provide a better title for this PR.

@GspikeHalo GspikeHalo changed the title Add love and clone Add Like and Clone Button Functionality with Database Enhancements for User Interaction Tracking Oct 12, 2024
Base automatically changed from add-clone-button to master October 12, 2024 17:22
@aglinxinyuan aglinxinyuan self-requested a review October 27, 2024 01:48
Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

LGTM!

@GspikeHalo GspikeHalo merged commit f3bcdb1 into master Oct 27, 2024
@GspikeHalo GspikeHalo deleted the add-love-button branch October 27, 2024 05:09
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
…r User Interaction Tracking (#2902)

**Propose:**
Add the like button while refactoring the clone button.

**Changes:**
1. Added two new tables to the database to track likes and clones. Use
16.sql to update the database.
2. Added a like button to the list item, which will be displayed in the
hub interface, but disabled when the user is not logged in.
3. Added a like button and a clone button to the detail page, both of
which will be disabled when the user is not logged in. The clone button
will be disabled in the detail preview interface

**Demo:**
**Old:**
list item before login:
![old list item before
login](https://github.com/user-attachments/assets/2729d307-2564-43cc-a852-1fb869633ae8)

detail page before login:
![old detail page before
login](https://github.com/user-attachments/assets/296ae4d0-11df-4486-9c7f-074fbc022a07)

list item after login:
![old list item after
login](https://github.com/user-attachments/assets/0923770c-2e1a-4122-a31f-6c5410f64334)

detail page after login:
![old detail page after
login](https://github.com/user-attachments/assets/92a0f08c-e1db-4aca-90a4-819595ba3bf6)

**New:**
list item before login:
![new hub list item before
login](https://github.com/user-attachments/assets/cb5e8ea8-75e0-4241-a1f4-fd3c46cefd7a)

detail page before login:
![new detail page before
login](https://github.com/user-attachments/assets/17c2ebac-9799-44aa-acb1-bd3670f1a9b8)

list item after login:
![new hub list item after
login](https://github.com/user-attachments/assets/7253cce9-ed85-4916-b686-dc286c71b4e9)

detail page after login:
![new detail page after
login](https://github.com/user-attachments/assets/356cb92e-7d5e-4a5c-895e-2135b7843301)

---------

Co-authored-by: Kyuho (Kyu) Oh <80994706+sixsage@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ddl-change Changes to the TexeraDB DDL starter-task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants