Skip to content

Twitter feature flags#1858

Merged
garrrikkotua merged 1 commit intomainfrom
feature/twitter-ff
Nov 20, 2023
Merged

Twitter feature flags#1858
garrrikkotua merged 1 commit intomainfrom
feature/twitter-ff

Conversation

@garrrikkotua
Copy link
Copy Markdown
Contributor

@garrrikkotua garrrikkotua commented Nov 20, 2023

Changes proposed ✍️

What

🤖[deprecated] Generated by Copilot at 572a883

Refactored the Twitter integration component to use a drawer and added feature flag and plan checks. Removed unused component and file. Added scale property to config.

🤖[deprecated] Generated by Copilot at 572a883

Sing, O Muse, of the skillful coder who refactored
The twitter-connect.vue component, making it more splendid
With a drawer that slides smoothly, hiding or revealing
The settings for the Twitter integration, a glorious feature.

Why

How

🤖[deprecated] Generated by Copilot at 572a883

  • Refactor twitter-connect.vue component to use a drawer instead of a modal and add feature flag and plan checks (link)
  • Remove unused twitter-connect-2.vue component and update config.js to remove its reference and add scale property to Twitter integration config (link, link, link)
  • Add twitter feature flag to featureFlag/index.js to enable or disable Twitter integration feature (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@garrrikkotua garrrikkotua merged commit 5ae8129 into main Nov 20, 2023
@garrrikkotua garrrikkotua deleted the feature/twitter-ff branch November 20, 2023 19:06
sausage-todd pushed a commit that referenced this pull request Dec 7, 2023
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.

1 participant