Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add cba flag #1976

Merged
merged 1 commit into from
Feb 20, 2024
Merged

add cba flag #1976

merged 1 commit into from
Feb 20, 2024

Conversation

iram-shehzadi
Copy link
Contributor

@iram-shehzadi iram-shehzadi commented Jan 16, 2024

What

This pull request introduces a feature that allows admins to control the visibility of CBA information in the logs through the use of a flag. Additionally, a user interface has been implemented specifically for superadmins, enabling them to toggle the CBA feature on and off.

Why

The motivation behind these changes is to facilitate a controlled rollout of the CBA feature to selected organisations. The key objectives are as follows:

  1. Gradual Rollout:

    • The introduction of the flag feature empowers superadmins to selectively enable or disable the CBA display in logs. This ensures a gradual rollout, allowing us to test and validate the feature in a controlled manner.
  2. Avoid Confusion:

    • By providing superadmins with a dedicated UI to manage the CBA feature, we prevent confusion among other admins who may not be using CBA yet.
  3. User Friendly Control:

    • The inclusion of a UI for superadmins makes the management of the CBA feature user friendly and accessible, providing them with a centralised control point.
  4. Customisation for Organisations:

    • Superadmins now have the flexibility to customise the CBA visibility based on the specific needs and readiness of different organisations.

This change aligns with our strategy to ensure a smooth and well managed integration of the CBA feature into our system, optimising user experience and minimising disruption.

Link to JIRA card (if applicable):
[GW-xxxx](https://technologyprogramme.atlassian.net/browse/GW-1267)

@iram-shehzadi iram-shehzadi changed the title Cba flag add cba flag Jan 16, 2024
Copy link
Contributor

@koetsier koetsier left a comment

Choose a reason for hiding this comment

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

some small changes

Copy link
Contributor

@koetsier koetsier left a comment

Choose a reason for hiding this comment

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

lgtm

@iram-shehzadi iram-shehzadi merged commit 331738c into master Feb 20, 2024
1 check passed
@iram-shehzadi iram-shehzadi deleted the cba-flag branch February 20, 2024 19:16
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.

2 participants