mirror of
https://github.com/discourse/discourse.git
synced 2026-03-03 22:05:52 +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 |
||
|---|---|---|
| .. | ||
| 20250304074934_backfill_api_key_scope_modes.rb | ||
| 20250305233449_populate_type_source_in_reviewable.rb | ||
| 20250513161753_rename_by_id_group.rb | ||
| 20250526063633_copy_add_groups_to_about_component_settings.rb | ||
| 20250702082232_deprecate_external_system_avatars_enabled.rb | ||
| 20250708031631_remove_flush_timings_secs_setting.rb | ||
| 20250709051949_disable_gravatar_enabled_if_unconfigured.rb | ||
| 20250710074447_clear_backup_frequency_if_disabled.rb | ||
| 20250804021210_drop_enable_experimental_sidebar_user_option.rb | ||
| 20250812033430_replace_null_with_zero_in_backup_frequency.rb | ||
| 20250821150220_remove_use_overhauled_theme_color_palette_setting.rb | ||
| 20250821155127_drop_dark_hex_from_color_scheme_color.rb | ||
| 20250821155615_drop_theme_color_scheme.rb | ||
| 20251029072625_delete_old_watched_precedence_setting.rb | ||
| 20251127001422_remove_orphaned_reviewable_claimed_topics.rb | ||
| 20260108044513_drop_imap_sync_logs.rb | ||
| 20260112082200_drop_reviewable_action_logs.rb | ||
