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

GitAuto: Make our sitemap dynamically updated using a library #172

Merged
merged 12 commits into from
Jan 5, 2025
7 changes: 7 additions & 0 deletions next-sitemap.config.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add this url https://www.npmjs.com/package/next-sitemap for future ref

Copy link
Author

@gitauto-for-dev gitauto-for-dev bot Jan 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolved your feedback! Looks good?

Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
// https://www.npmjs.com/package/next-sitemap
const { NEXT_PUBLIC_SITE_URL } = require('@/config');
module.exports = {
siteUrl: NEXT_PUBLIC_SITE_URL,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, use NEXT_PUBLIC_SITE_URL from env simply not from @/config

Copy link
Author

@gitauto-for-dev gitauto-for-dev bot Jan 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolved your feedback! Looks good?

generateRobotsTxt: true,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it false

Copy link
Author

@gitauto-for-dev gitauto-for-dev bot Jan 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolved your feedback! Looks good?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it false

Copy link
Author

@gitauto-for-dev gitauto-for-dev bot Jan 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolved your feedback! Looks good?

// Additional configurations as needed
};
2 changes: 2 additions & 0 deletions package.json
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add next-sitemap.config.js because it is required.

Copy link
Author

@gitauto-for-dev gitauto-for-dev bot Jan 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resolved your feedback! Looks good?

Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
"scripts": {
"dev": "next dev",
"build": "next build",
"postbuild": "next-sitemap",
"start": "next start",
"lint": "next lint"
},
Expand Down Expand Up @@ -37,6 +38,7 @@
"jsonwebtoken": "^9.0.2",
"next": "^15.0.3",
"next-auth": "^4.24.10",
"next-sitemap": "^4.2.3",
"playwright": "^1.49.0",
"posthog-js": "^1.194.1",
"react": "^18.3.1",
Expand Down
Loading