Skip to content

Add view count to workflow #2930

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
Merged
GspikeHalo merged 72 commits into from
Oct 28, 2024
Merged

Add view count to workflow #2930

GspikeHalo merged 72 commits into from
Oct 28, 2024

Conversation

@sixsage
Copy link
Contributor

@sixsage sixsage commented Oct 14, 2024 •
edited
Loading

Purpose

Add view count to workflows

Changes

  1. Create a new table, workflow view count, in texera ddl. A new file 17.sql was also added for the table.
  2. The view counts for a workflow increment whenever a user accesses the hub-workflow-detail page or the workflow workspace for that particular workflow.
  3. The API call to increment the view is limited to be called only once per second using throttleTime

Demo

List item before
image
List item after
image
hub-workflow-detail page before
image
hub-workflow-detail page after
Default display
image
After clicking on button (precise display)
image

GspikeHalo and others added 30 commits October 1, 2024 23:32
@GspikeHalo
Add clone & edit button on detail page
cddb029
@GspikeHalo
Merge branch 'master' into add-clone-button
5ab607f
@GspikeHalo
Merge branch 'master' into add-clone-button
a37b759
@GspikeHalo
Modify the clone button style
f440ec1
@GspikeHalo
fmt fix
e84ee16
@GspikeHalo
Add two new tables: workflow_user_clones and workflow_user_likes.
4a3ae5a
@GspikeHalo
add like button on list item
5ebe2c2
@GspikeHalo
add live button in detail page
2347c26
@GspikeHalo
fmt fix
09c9649
@GspikeHalo
quick fix
43fdf26
@GspikeHalo
Implement statistics for cloneworkflow and add counts for like and cl…
01f0887 
…one to the detail interface.
@GspikeHalo
Disable the copy button for the owner preview detail interface
3c04c01
@GspikeHalo
fmt fix
eb29b7b
@GspikeHalo
quick fix
298f1e4
@sixsage
add view table
7942f78
@GspikeHalo
Merge branch 'master' into add-clone-button
479b8f0
@GspikeHalo
Merge branch 'master' into add-clone-button
6697f94
@GspikeHalo
Merge branch 'master' into add-clone-button
e5205cb
@GspikeHalo
Merge branch 'add-clone-button' into add-love-button
7a14364
@GspikeHalo
add like count in list item
5ec462b
@sixsage
update ddl
df45e09
@sixsage
Merge branch 'add-love-button' into add-views
005d1c8
@GspikeHalo
enable the like button on hub list item when user is not logged in
b34c9f1
@GspikeHalo
display but disable like and clone button in detail page
7684f1c
@GspikeHalo
A dialog box is displayed when the clone is successful.
68259bc
@GspikeHalo
fmt fix
ef7bb9e
@GspikeHalo
Merge branch 'master' into add-clone-button
b62f176
@GspikeHalo
Merge branch 'add-clone-button' into add-love-button
cdd4764
@GspikeHalo
Merge branch 'master' into add-clone-button
0174e78
@GspikeHalo
Merge branch 'add-clone-button' into add-love-button
b8c6fe8
Base automatically changed from add-love-button to master October 27, 2024 05:09
@sixsage sixsage self-assigned this Oct 27, 2024
@sixsage sixsage added ddl-change Changes to the TexeraDB DDL starter-task gui frontend Changes related to the frontend GUI labels Oct 28, 2024
@sixsage sixsage marked this pull request as ready for review October 28, 2024 10:19
@sixsage sixsage requested a review from aglinxinyuan October 28, 2024 10:19
aglinxinyuan
aglinxinyuan approved these changes Oct 28, 2024
View reviewed changes
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 166e77c into master Oct 28, 2024
@GspikeHalo GspikeHalo deleted the add-views branch October 28, 2024 19:52
@bobbai00 bobbai00 mentioned this pull request Nov 2, 2024
Merged
bobbai00 added a commit that referenced this pull request Nov 2, 2024
@bobbai00
Update jooq-generated codes in DAO under micro-services (#2989)
4cfd0d6 
Since texera_ddl.sql is updated in #2930 , and #2960 is merged after it,
the jooq-generated classes in `micro-services/dao` is not updated. This
PR make the `micro-services/dao` up-to-date.
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
@sixsage @GspikeHalo
@aglinxinyuan @PurelyBlank
Add view count to workflow (#2930)
d805fe3 
## Purpose
Add view count to workflows
## Changes
1. Create a new table, workflow view count, in texera ddl. A new file
17.sql was also added for the table.
2. The view counts for a workflow increment whenever a user accesses the
hub-workflow-detail page or the workflow workspace for that particular
workflow.
3. The API call to increment the view is limited to be called only once
per second using throttleTime
## Demo
List item before

![image](https://github.com/user-attachments/assets/ede5e665-e50c-496e-a239-f270017f7407)
List item after

![image](https://github.com/user-attachments/assets/0e9274a8-d9c9-4165-8c33-bab204e74602)
hub-workflow-detail page before

![image](https://github.com/user-attachments/assets/e6e33e33-724f-4e52-a1e5-34f45f23ade9)
hub-workflow-detail page after
Default display

![image](https://github.com/user-attachments/assets/5aab223e-f419-43bf-9cfc-db72681d67d1)
After clicking on button (precise display)

![image](https://github.com/user-attachments/assets/486569e2-d19e-4d43-a624-2a5001331e41)

---------

Co-authored-by: gspikehalo <2318002579@qq.com>
Co-authored-by: GspikeHalo <109092664+GspikeHalo@users.noreply.github.com>
Co-authored-by: Xinyuan Lin <xinyual3@uci.edu>
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
@bobbai00 @PurelyBlank
Update jooq-generated codes in DAO under micro-services (#2989)
d1ab39a 
Since texera_ddl.sql is updated in #2930 , and #2960 is merged after it,
the jooq-generated classes in `micro-services/dao` is not updated. This
PR make the `micro-services/dao` up-to-date.
@Yicong-Huang Yicong-Huang removed the gui label Jul 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aglinxinyuan aglinxinyuan aglinxinyuan approved these changes

Assignees

@sixsage sixsage

@GspikeHalo GspikeHalo

Labels

ddl-change Changes to the TexeraDB DDL frontend Changes related to the frontend GUI starter-task

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sixsage @aglinxinyuan @Yicong-Huang @GspikeHalo