Skip to content

Remove rating and review system from Tool Store#609

Merged
brendanx67 merged 4 commits intorelease25.11-SNAPSHOTfrom
25.11_fb_toolstore-cleanup
Mar 13, 2026
Merged

Remove rating and review system from Tool Store#609
brendanx67 merged 4 commits intorelease25.11-SNAPSHOTfrom
25.11_fb_toolstore-cleanup

Conversation

@brendanx67
Copy link
Collaborator

Summary

  • Removed the rating/review submission UI (broken layout, rarely used)
  • Deleted RatingManager.java, Rating.java, SubmitRatingAction, DeleteRatingAction
  • Cleaned up rating display, CSS, and JS from both tool detail and tool list pages
  • Fixed InsertAction NavTree to use flat breadcrumb structure
  • Rating table remains in schema — may need a schema update script to drop it

Test plan

  • Verified tool detail page renders without review panel
  • Verified tool list page renders without rating display
  • Module compiles cleanly

Co-Authored-By: Claude noreply@anthropic.com

* Deleted RatingManager.java and Rating.java model class
* Removed SubmitRatingAction and DeleteRatingAction from controller
* Removed review forms, rating display, and rating CSS from both JSPs
* Removed initRatingSlider from functions.js
* Fixed InsertAction NavTree to use flat breadcrumb structure

Co-Authored-By: Claude <noreply@anthropic.com>
@vagisha vagisha marked this pull request as ready for review March 13, 2026 15:59
@vagisha vagisha requested a review from labkey-jeckels March 13, 2026 15:59
Copy link
Contributor

@labkey-jeckels labkey-jeckels left a comment

Choose a reason for hiding this comment

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

I think star_full15x15.png, star_empty15x15.png, star_full20x20.png, and star_empty20x20.png can be deleted as well.

@vagisha
Copy link
Collaborator

vagisha commented Mar 13, 2026

I think star_full15x15.png, star_empty15x15.png, star_full20x20.png, and star_empty20x20.png can be deleted as well.

Yes. I just deleted them. Thanks.

@brendanx67 brendanx67 merged commit 389a437 into release25.11-SNAPSHOT Mar 13, 2026
5 checks passed
@brendanx67 brendanx67 deleted the 25.11_fb_toolstore-cleanup branch March 13, 2026 19:21
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.

3 participants