Something went wrong while fetching comments. Please try again.

Adds global CTA banner

Review changes
Download
Patches
Plain diff

Merged Adds global CTA banner

ar/global-cta-context-wip into main

Overview 1
Commits 12
Pipelines 0
Changes 9

Merged Adds global CTA banner

Administratorrequested to merge

ar/global-cta-context-wip into main Jun 9, 2021

Overview 1
Commits 12
Pipelines 0
Changes 9

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.