Revert "fix: Only show GH config banner when no welcome banner (#3680)"

This reverts commit dfb4df6.
codecov · Jan 24, 2025 · 0cd73f1 · 0cd73f1
1 parent dfb4df6
commit 0cd73f1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 17 deletions.
diff --git a/src/shared/ListRepo/ListRepo.jsx b/src/shared/ListRepo/ListRepo.jsx
@@ -60,8 +60,7 @@ function ListRepo({ canRefetch, hasGhApp }) {
 
   return (
     <>
-      {/* we only want one of these two banners to show at a time */}
-      {!hasGhApp && !showDemoAlert && <GithubConfigBanner />}
+      {!hasGhApp && <GithubConfigBanner />}
       <OrgControlTable
         searchValue={params.search}
         repoDisplay={repoDisplay}

diff --git a/src/shared/ListRepo/ListRepo.test.tsx b/src/shared/ListRepo/ListRepo.test.tsx
@@ -303,7 +303,7 @@ describe('ListRepo', () => {
   })
 
   describe('user does not have gh app installed', () => {
-    it('displays github app config banner if showDemoAlert is false', async () => {
+    it('displays github app config banner', async () => {
       setup({})
       render(<ListRepo canRefetch hasGhApp={false} />, {
         wrapper: wrapper({
@@ -312,21 +312,9 @@ describe('ListRepo', () => {
         }),
       })
 
-      const banner = await screen.findByText("Codecov's GitHub app")
-      return expect(banner).toBeInTheDocument()
-    })
-    it('does not display github app config banner if showDemoAlert is true', async () => {
-      setup({})
-      render(<ListRepo canRefetch hasGhApp={false} />, {
-        wrapper: wrapper({
-          url: '/gh/janedoe?source=onboarding',
-          path: '/:provider/:owner',
-        }),
-      })
-
       await waitFor(() => {
-        const banner = screen.queryByText("Codecov's GitHub app")
-        expect(banner).not.toBeInTheDocument()
+        const banner = screen.getByText("Codecov's GitHub app")
+        return expect(banner).toBeInTheDocument()
       })
     })
   })