Financial-Times / x-dash

:x::heavy_minus_sign::newspaper: shared front-end components for FT.com and the FT Apps
https://financial-times.github.io/x-dash
38 stars 6 forks source link

chore: Remove isArticleSharingUxUpdates stale code #715

Closed Nazehs closed 1 year ago

Nazehs commented 1 year ago

This pull request aims to remove unused code and flags associated with the isArticleSharingUxUpdates feature in the x-dash codebases. The feature has been inactive for the past two years, and it appears to be unnecessary in the current context.

Upon thorough review of the code and discussions within the thread, it has been determined that the isArticleSharingUxUpdatesflag and its associated code can be safely removed. Deleting this unused code will help declutter the codebase and improve maintainability.

This pull request includes the following changes:

Removal of the isArticleSharingUxUpdates flag declaration. Elimination of the code blocks related to the isArticleSharingUxUpdates feature. Cleaning up any references or dependencies on the removed code. Please review the changes to ensure that the removal of the code and flags is appropriate and does not impact any existing functionality.

Please refer to the thread for additional context and any relevant links. Slack Thread

Jira Ticket