Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Commit

Permalink
Merge pull request #255 from githubnext/jaked/blocks-completed-banner
Browse files Browse the repository at this point in the history
banner on all pages with completed message
  • Loading branch information
jaked authored Nov 14, 2023
2 parents 3ac4db7 + 5c2384c commit 1fe7ce0
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 6 deletions.
2 changes: 1 addition & 1 deletion components/full-page-loader.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
export function FullPageLoader() {
return (
<div className="h-screen flex items-center justify-center">
<div className="h-full flex items-center justify-center">
<img src="/next-octocat.svg" alt="Logo" className="w-32 animate-pulse" />
</div>
);
Expand Down
2 changes: 1 addition & 1 deletion components/repo-detail/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ export function RepoDetailInner(props: RepoDetailInnerProps) {
const { visible: historyPane } = useHistoryPane();

return (
<div className="flex flex-col h-screen overflow-hidden">
<div className="flex flex-col h-full overflow-hidden">
<Head>
<title>
{/* mimicking github.com's title */}
Expand Down
20 changes: 16 additions & 4 deletions pages/_app.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,8 @@ function App({ Component, pageProps: { session, ...pageProps } }) {
if (process.env.NODE_ENV !== "production") return;
const appInsights = new ApplicationInsights({
config: {

// this isn't a high-security key, see https://stackoverflow.com/questions/54535275/what-will-happen-if-applicationinsights-instrumentationkey-gets-stolen
connectionString:
`InstrumentationKey=96006ad9-5042-466e-b5c9-641be3a9e13f;IngestionEndpoint=/api/telemetry/ingest;LiveEndpoint=/api/telemetry/live`,
connectionString: `InstrumentationKey=96006ad9-5042-466e-b5c9-641be3a9e13f;IngestionEndpoint=/api/telemetry/ingest;LiveEndpoint=/api/telemetry/live`,
},
});
appInsights.loadAppInsights();
Expand Down Expand Up @@ -78,7 +76,21 @@ function App({ Component, pageProps: { session, ...pageProps } }) {
session={session}
refetchInterval={5 * 60}
>
<Component {...pageProps} />
<div className="h-screen flex flex-col">
<div className="text-center p-4 bg-yellow-200">
The Blocks technical preview is shutting down on December
15th, 2023.{" "}
<a
href="https://gist.github.com/idan/325676d192b32f169b032fde2d866c2c#github-next--technical-preview-sunsets"
className="underline"
>
Learn more about why we're sunsetting Blocks.
</a>
</div>
<div className="flex-1 overflow-auto">
<Component {...pageProps} />
</div>
</div>
</SessionProvider>
</BaseStyles>
</ThemeProvider>
Expand Down

0 comments on commit 1fe7ce0

Please sign in to comment.