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

Revert Live Now Overlay removal #59

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,6 @@ yarn-error.log*

# vercel
.vercel

# Sentry Config File
.env.sentry-build-plugin
8 changes: 3 additions & 5 deletions components/HomeFeed/HomeFeed.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ import { Box, CardGrid, Heading, Section, Text } from 'ui-kit';
import { getMediaSource, getSlugFromURL } from 'utils';
import Styled from './HomeFeed.styles';
import useLiveStreams from 'hooks/useLiveStreams';
import * as Sentry from '@sentry/nextjs';

function FullLengthSermon(props = {}) {
const router = useRouter();
Expand All @@ -30,11 +31,8 @@ function FullLengthSermon(props = {}) {
const { liveStreams } = useLiveStreams();
const liveContent = liveStreams?.[0]?.contentItem;
const livestreamUrl = liveStreams?.[0]?.webViewUrl;
// TODO: we have a bug with this we need to fix
// https://linear.app/differential/issue/APO-212/long-hollow-watch-live-now-overlay-is-still-showing-on-the-home-page
//const LIVE = !!liveStreams?.[0]?.isLive;
const LIVE = false;

const LIVE = !!liveStreams?.[0]?.isLive;
Sentry.captureMessage(`LIVE: ${JSON.stringify(LIVE)}`);
const clips =
props.sermon?.childContentItemsConnection?.edges
?.map(({ node }) => node)
Expand Down
20 changes: 11 additions & 9 deletions hooks/useLiveStreams.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import { gql, useQuery } from '@apollo/client';
import { formatDistanceToNow } from 'date-fns';
import * as Sentry from '@sentry/nextjs';

export const GET_LIVE_STREAMS = gql`
query getLiveStreams {
Expand Down Expand Up @@ -39,17 +40,18 @@ function useLiveStreams(options = {}) {
const query = useQuery(GET_LIVE_STREAMS, {
...options,
cachePolicy: 'network-only',
errorPolicy: 'ignore',
});
Sentry.captureMessage(`useLiveStreams: ${JSON.stringify(query?.data)}`);
const firstStream = query?.data?.liveStreams?.[0];
let prettyCountdown = '';
// TODO turn this back on when livestream cache is working properly
//if (firstStream?.isLive) prettyCountdown = '• LIVE NOW';
//else if (!firstStream?.eventStartTime) prettyCountdown = '';
//else
//prettyCountdown = `• LIVE ${formatDistanceToNow(
//new Date(firstStream?.eventStartTime),
//{ addSuffix: true }
//).toUpperCase()}`;
let prettyCountdown;
if (firstStream?.isLive) prettyCountdown = '• LIVE NOW';
else if (!firstStream?.eventStartTime) prettyCountdown = '';
else
prettyCountdown = `• LIVE ${formatDistanceToNow(
new Date(firstStream?.eventStartTime),
{ addSuffix: true }
).toUpperCase()}`;

return {
prettyCountdown,
Expand Down
13 changes: 13 additions & 0 deletions instrumentation.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
import * as Sentry from '@sentry/nextjs';

export async function register() {
if (process.env.NEXT_RUNTIME === 'nodejs') {
await import('./sentry.server.config');
}

if (process.env.NEXT_RUNTIME === 'edge') {
await import('./sentry.edge.config');
}
}

export const onRequestError = Sentry.captureRequestError;
43 changes: 43 additions & 0 deletions next.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -326,3 +326,46 @@ module.exports = {
];
},
};


// Injected content via Sentry wizard below

const { withSentryConfig } = require("@sentry/nextjs");

module.exports = withSentryConfig(
module.exports,
{
// For all available options, see:
// https://github.com/getsentry/sentry-webpack-plugin#options

org: "differential-ka",
project: "longhollow-web",

// Only print logs for uploading source maps in CI
silent: !process.env.CI,

// For all available options, see:
// https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/

// Upload a larger set of source maps for prettier stack traces (increases build time)
widenClientFileUpload: true,

// Uncomment to route browser requests to Sentry through a Next.js rewrite to circumvent ad-blockers.
// This can increase your server load as well as your hosting bill.
// Note: Check that the configured route will not match with your Next.js middleware, otherwise reporting of client-
// side errors will fail.
// tunnelRoute: "/monitoring",

// Hides source maps from generated client bundles
hideSourceMaps: true,

// Automatically tree-shake Sentry logger statements to reduce bundle size
disableLogger: true,

// Enables automatic instrumentation of Vercel Cron Monitors. (Does not yet work with App Router route handlers.)
// See the following for more information:
// https://docs.sentry.io/product/crons/
// https://vercel.com/docs/cron-jobs
automaticVercelMonitors: true,
}
);
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
},
"dependencies": {
"@apollo/client": "^3.2.5",
"@sentry/nextjs": "^8",
"@storybook/cli": "^6.1.11",
"@styled-system/theme-get": "^5.1.2",
"algoliasearch": "^4.9.0",
Expand Down
17 changes: 17 additions & 0 deletions pages/_error.jsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
import * as Sentry from '@sentry/nextjs';
import Error from 'next/error';

const CustomErrorComponent = props => {
return <Error statusCode={props.statusCode} />;
};

CustomErrorComponent.getInitialProps = async contextData => {
// In case this is running in a serverless function, await this in order to give Sentry
// time to send the error before the lambda exits
await Sentry.captureUnderscoreErrorException(contextData);

// This will contain the status code of the response
return Error.getInitialProps(contextData);
};

export default CustomErrorComponent;
5 changes: 5 additions & 0 deletions pages/api/sentry-example-api.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
// A faulty API route to test Sentry's error monitoring
export default function handler(_req, res) {
throw new Error("Sentry Example API Route Error");
res.status(200).json({ name: "John Doe" });
}
77 changes: 77 additions & 0 deletions pages/sentry-example-page.jsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,77 @@
import Head from "next/head";
import * as Sentry from "@sentry/nextjs";

export default function Page() {
return (
<div>
<Head>
<title>Sentry Onboarding</title>
<meta name="description" content="Test Sentry for your Next.js app!" />
</Head>

<main
style={{
minHeight: "100vh",
display: "flex",
flexDirection: "column",
justifyContent: "center",
alignItems: "center",
}}
>
<h1 style={{ fontSize: "4rem", margin: "14px 0" }}>
<svg
style={{
height: "1em",
}}
xmlns="http://www.w3.org/2000/svg"
viewBox="0 0 200 44"
>
<path
fill="currentColor"
d="M124.32,28.28,109.56,9.22h-3.68V34.77h3.73V15.19l15.18,19.58h3.26V9.22h-3.73ZM87.15,23.54h13.23V20.22H87.14V12.53h14.93V9.21H83.34V34.77h18.92V31.45H87.14ZM71.59,20.3h0C66.44,19.06,65,18.08,65,15.7c0-2.14,1.89-3.59,4.71-3.59a12.06,12.06,0,0,1,7.07,2.55l2-2.83a14.1,14.1,0,0,0-9-3c-5.06,0-8.59,3-8.59,7.27,0,4.6,3,6.19,8.46,7.52C74.51,24.74,76,25.78,76,28.11s-2,3.77-5.09,3.77a12.34,12.34,0,0,1-8.3-3.26l-2.25,2.69a15.94,15.94,0,0,0,10.42,3.85c5.48,0,9-2.95,9-7.51C79.75,23.79,77.47,21.72,71.59,20.3ZM195.7,9.22l-7.69,12-7.64-12h-4.46L186,24.67V34.78h3.84V24.55L200,9.22Zm-64.63,3.46h8.37v22.1h3.84V12.68h8.37V9.22H131.08ZM169.41,24.8c3.86-1.07,6-3.77,6-7.63,0-4.91-3.59-8-9.38-8H154.67V34.76h3.8V25.58h6.45l6.48,9.2h4.44l-7-9.82Zm-10.95-2.5V12.6h7.17c3.74,0,5.88,1.77,5.88,4.84s-2.29,4.86-5.84,4.86Z M29,2.26a4.67,4.67,0,0,0-8,0L14.42,13.53A32.21,32.21,0,0,1,32.17,40.19H27.55A27.68,27.68,0,0,0,12.09,17.47L6,28a15.92,15.92,0,0,1,9.23,12.17H4.62A.76.76,0,0,1,4,39.06l2.94-5a10.74,10.74,0,0,0-3.36-1.9l-2.91,5a4.54,4.54,0,0,0,1.69,6.24A4.66,4.66,0,0,0,4.62,44H19.15a19.4,19.4,0,0,0-8-17.31l2.31-4A23.87,23.87,0,0,1,23.76,44H36.07a35.88,35.88,0,0,0-16.41-31.8l4.67-8a.77.77,0,0,1,1.05-.27c.53.29,20.29,34.77,20.66,35.17a.76.76,0,0,1-.68,1.13H40.6q.09,1.91,0,3.81h4.78A4.59,4.59,0,0,0,50,39.43a4.49,4.49,0,0,0-.62-2.28Z"
></path>
</svg>
</h1>

<p>Get started by sending us a sample error:</p>
<button
type="button"
style={{
padding: "12px",
cursor: "pointer",
backgroundColor: "#AD6CAA",
borderRadius: "4px",
border: "none",
color: "white",
fontSize: "14px",
margin: "18px",
}}
onClick={async () => {
await Sentry.startSpan({
name: 'Example Frontend Span',
op: 'test'
}, async () => {
const res = await fetch("/api/sentry-example-api");
if (!res.ok) {
throw new Error("Sentry Example Frontend Error");
}
});
}}
>
Throw error!
</button>

<p>
Next, look for the error on the{" "}
<a href="https://differential-ka.sentry.io/issues/?project=4508020008615936">Issues Page</a>.
</p>
<p style={{ marginTop: "24px" }}>
For more information, see{" "}
<a href="https://docs.sentry.io/platforms/javascript/guides/nextjs/">
https://docs.sentry.io/platforms/javascript/guides/nextjs/
</a>
</p>
</main>
</div>
);
}
12 changes: 12 additions & 0 deletions sentry.client.config.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// This file configures the initialization of Sentry on the client.
// The config you add here will be used whenever a users loads a page in their browser.
// https://docs.sentry.io/platforms/javascript/guides/nextjs/

import * as Sentry from "@sentry/nextjs";

Sentry.init({
dsn: "https://[email protected]/4508020008615936",

// Setting this option to true will print useful information to the console while you're setting up Sentry.
debug: false,
});
13 changes: 13 additions & 0 deletions sentry.edge.config.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// This file configures the initialization of Sentry for edge features (middleware, edge routes, and so on).
// The config you add here will be used whenever one of the edge features is loaded.
// Note that this config is unrelated to the Vercel Edge Runtime and is also required when running locally.
// https://docs.sentry.io/platforms/javascript/guides/nextjs/

import * as Sentry from "@sentry/nextjs";

Sentry.init({
dsn: "https://[email protected]/4508020008615936",

// Setting this option to true will print useful information to the console while you're setting up Sentry.
debug: false,
});
12 changes: 12 additions & 0 deletions sentry.server.config.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// This file configures the initialization of Sentry on the server.
// The config you add here will be used whenever the server handles a request.
// https://docs.sentry.io/platforms/javascript/guides/nextjs/

import * as Sentry from "@sentry/nextjs";

Sentry.init({
dsn: "https://[email protected]/4508020008615936",

// Setting this option to true will print useful information to the console while you're setting up Sentry.
debug: false,
});
Loading