mirror of
https://github.com/discourse/discourse.git
synced 2026-03-03 23:54:20 +08:00
53d799eb8b
Following PR #36711 which enabled the refreshed review UI for all users, this commit removes the old feature flag infrastructure and cleans up legacy code that is no longer needed. Backend Changes: - Removed ReviewableActionLog model and its spec entirely - Removed CalculateFinalStatusFromLogs service and spec - Removed site settings: force_old_reviewable_ui, reviewable_old_moderator_actions - Simplified ReviewableActionBuilder — stripped out legacy action-building methods (build_user_actions_bundle, build_post_actions_bundle, build_new_separated_actions) - Cleaned up reviewable models (ReviewableFlaggedPost, ReviewablePost, ReviewableQueuedPost, ReviewableUser, Chat::ReviewableMessage, ReviewablePostVotingComment) — removed unused/legacy action definitions - Removed legacy specs for action builder, action logs, flagged post actions, post actions, user actions, and status-from-logs - Updated system tests and page objects to reflect the new UI structure Frontend Changes: - Deleted legacy components: reviewable-item.gjs, reviewable-user.gjs, review-index-legacy.gjs - Renamed reviewable-refresh/ → reviewable/ — moved all sub-components (created-by, flagged-post, item, post, queued-post, topic-link, user, etc.) out of the refresh directory into the canonical reviewable/ namespace - Simplified reviewable/item.gjs — removed feature flag conditionals and legacy code paths - Cleaned up review/index.gjs and review/show.gjs templates — removed branching between old/new UI - Updated plugin components (chat, AI, post-voting) to import from reviewable/ instead of reviewable-refresh/ - Removed acceptance tests (review-test.js) replaced by system tests - Renamed and updated integration tests from reviewable-refresh/* to reviewable/* Plan for next PRs: - Move plugins from Pages::RefreshedReview to Pages::Review - Move plugins to import from `reviewable/` and not `refreshed-reviewable/` - Move reviewable-user.js import in plugin to use `reviewable/user.js` - Remove unused settings like `reviewable_old_moderator_actions` from plugins - Delete `Pages::RefreshedReview` - Delete `reviewable-refresh/` directory - Delete `reviewable-user.js` component - Delete `reviewable_old_moderator_actions` site setting Plugins PRs: - https://github.com/discourse/discourse-akismet/pull/203 - https://github.com/discourse/discourse-antivirus/pull/98 - https://github.com/discourse/discourse-category-experts/pull/223 |
||
|---|---|---|
| .. | ||
| upcoming_changes | ||
| vanilla_import | ||
| 2000x2000.png | ||
| animated.gif | ||
| animated.webp | ||
| bad-xml-icon-sprite.svg | ||
| cropped.png | ||
| custom-theme-component-icon-sprite.svg | ||
| custom-theme-icon-sprite.svg | ||
| downsized.png | ||
| fake.jpg | ||
| fake.long-FileExtension | ||
| fake.not_image | ||
| huge.jpg | ||
| image.svg | ||
| image_no_extension | ||
| large_and_unoptimized.png | ||
| large_icon_correct.png | ||
| large_icon_incorrect.png | ||
| logo-dev.png | ||
| logo.jpg | ||
| logo.png | ||
| massive.svg | ||
| not_an_image | ||
| png_as.bin | ||
| pngquant.png | ||
| resized.png | ||
| should_be_jpeg.heic | ||
| should_be_jpeg.png | ||
| smallest.ico | ||
| smallest.png | ||
| svg.png | ||
| tiff_as.bin | ||
| tiny.svg | ||
| webp_as.bin | ||
| zero_sized.svg | ||
