Skip to content

LG-11569: Add feature flag for reorienting SDK capture#9624

Merged
night-jellyfish merged 2 commits intomainfrom
brittany/lg-11569-reorient-sdk-capture-feature-flag
Nov 22, 2023
Merged

LG-11569: Add feature flag for reorienting SDK capture#9624
night-jellyfish merged 2 commits intomainfrom
brittany/lg-11569-reorient-sdk-capture-feature-flag

Conversation

@night-jellyfish
Copy link
Contributor

🎫 Ticket

LG-11569

🛠 Summary of changes

Adding a feature flag to control whether users see the horizontal or vertical version of the SDK. This change just sets up the feature flag for future work.

I used this format for the flag because we want to be able to split the
number of users who see horizontal or vertical.

📜 Testing Plan

  • Log into the Rails console
  • Look for IdentityConfig.store.doc_auth_sdk_capture_orientation
  • Make sure you see: {:horizontal=>100, :vertical=>0}

👀 Screenshots

If relevant, include a screenshot or screen capture of the changes.

Before: image
After: image

@night-jellyfish night-jellyfish requested review from a team and dawei-nava and removed request for a team November 20, 2023 16:50
Copy link
Contributor

@dawei-nava dawei-nava left a comment

Choose a reason for hiding this comment

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

LGTM

Brittany Greaner added 2 commits November 22, 2023 11:09
I used this format for the flag because we want to be able to split the
number of users who see horizontal or vertical.
@night-jellyfish night-jellyfish force-pushed the brittany/lg-11569-reorient-sdk-capture-feature-flag branch from c5205fe to 8ab98fd Compare November 22, 2023 19:09
@night-jellyfish night-jellyfish merged commit abe6983 into main Nov 22, 2023
@night-jellyfish night-jellyfish deleted the brittany/lg-11569-reorient-sdk-capture-feature-flag branch November 22, 2023 20:11
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