Adds global CTA banner (!21950) · Merge requests · Administrator / sourcegraph

Something went wrong while fetching comments. Please try again.

Merged Administrator requested to merge ar/global-cta-context-wip into main 4 years ago

Created by: artemruts

Description

We need to show two CTA banners to the users that have a GitHub code host connection but it's missing the repo scope. This prevents users to access their private repositories.

These changes add a global dismissible banner for user that is:

on .com
enrolled in private code
is missing "private" scope for GitHub

React context was used here since I had "communicate" with two banners that are in a very different places in the render tree. I felt that passing props would create more problems and make the code less maintainable.

Note: this is a temporarily solution. CTA banners will be removed once the Core app team confirms that all of the users with GitHub code host connection have the repo scope.