Skip to content

Commit

Permalink
Merge branch 'master' into fix/nextjs-app-directories
Browse files Browse the repository at this point in the history
  • Loading branch information
Lms24 authored Sep 13, 2023
2 parents 6dab055 + 8dedd81 commit 9e1900c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
## Unreleased

- enh(android): Show link to issues page after setup is complete (#448)
- fix(nextjs): Add selfhosted url in `next.config.js` (#438)
- fix(nextjs): Create necessary directories in app router (#439)

Work in this release contributed by @andreysam. Thank you for your contributions!
Expand Down
2 changes: 2 additions & 0 deletions src/nextjs/nextjs-wizard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,8 @@ export async function runNextjsWizard(options: WizardOptions): Promise<void> {
const sentryWebpackOptionsTemplate = getNextjsWebpackPluginOptionsTemplate(
selectedProject.organization.slug,
selectedProject.slug,
selfHosted,
sentryUrl,
);
const sentryBuildOptionsTemplate = getNextjsSentryBuildOptionsTemplate();

Expand Down
5 changes: 3 additions & 2 deletions src/nextjs/templates.ts
Original file line number Diff line number Diff line change
@@ -1,16 +1,17 @@
export function getNextjsWebpackPluginOptionsTemplate(
orgSlug: string,
projectSlug: string,
selfHosted: boolean,
url: string,
): string {
return `{
// For all available options, see:
// https://github.com/getsentry/sentry-webpack-plugin#options
// Suppresses source map uploading logs during build
silent: true,
org: "${orgSlug}",
project: "${projectSlug}",
project: "${projectSlug}",${selfHosted ? `\n url: "${url}"` : ''}
}`;
}

Expand Down

0 comments on commit 9e1900c

Please sign in to comment.