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

Add terms and privacy pages #131 #178

Merged
merged 3 commits into from
Jul 16, 2023
Merged

Add terms and privacy pages #131 #178

merged 3 commits into from
Jul 16, 2023

Conversation

samanyougarg
Copy link
Member

@samanyougarg samanyougarg commented Jul 16, 2023

Implement #131

Summary by OpenAI

New Feature:

  • Added new Privacy Policy and Terms of Service pages to the application.
  • Introduced a PrivacyTermsBanner component that displays a banner with a title and last updated date.

🎉 With terms and privacy, we take a stand, 📜

New pages in our app, aren't they grand? 🌟

A banner so bright, with date and title at hand, 🗓️

Celebrate these changes, for they are truly well planned! 🎊

@vercel
Copy link

vercel bot commented Jul 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bg-frontend ❌ Failed (Inspect) Jul 16, 2023 7:40pm
bhagavad-gita ✅ Ready (Inspect) Visit Preview Jul 16, 2023 7:40pm

@vercel vercel bot temporarily deployed to Preview – bg-frontend July 16, 2023 19:05 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Jul 16, 2023

🤖 OpenAI

The changes introduce new components for Privacy Policy and Terms of Service pages, along with a shared banner component. These additions provide users with necessary legal information and enhance the overall user experience.

Files Summary
app/privacy-policy/layout.tsx, app/terms-of-service/layout.tsx New Layout.tsx files are added that export React components. They use ReactNode as the type for the children prop and render the children within a HomeLayout component.
app/privacy-policy/page.tsx, app/terms-of-service/page.tsx New page components for the Privacy Policy and Terms of Service have been added, containing descriptive text.
components/Shared/PrivacyTermsBanner.tsx A new shared component PrivacyTermsBanner is introduced. It displays a banner with a title and last updated date, using an image from the next/image package as the background.

Chat with 🤖 OpenAI Bot (@openai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @openai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Ignoring further reviews

  • Type @openai: ignore anywhere in the PR description to ignore further reviews from the bot.

Files not summarized due to errors (2)

Failed to summarize

  • app/privacy-policy/page.tsx (diff tokens exceeds limit)
  • app/terms-of-service/page.tsx (diff tokens exceeds limit)

In the recent run, only the files that changed from the base of the PR and between edbe7763e4fbea7bd970375f84295e995cc8d166 and dffde340dbb840ded05cfbe96fe9a78fd46b2909 commits were reviewed.

Comment on lines +8 to +14
export default function TermsOfService() {
return (
<>
<PrivacyTermsBanner
title={metadata.title as string}
lastUpdated="2023-07-17"
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

🤖 OpenAI

The lastUpdated prop in the PrivacyTermsBanner component is hardcoded to a specific date. This could lead to outdated information being displayed if the terms of service are updated but this date is not changed. Consider storing this date in a more centralized location or retrieving it from a backend service, if available.

Suggested change
export default function TermsOfService() {
return (
<>
<PrivacyTermsBanner
title={metadata.title as string}
lastUpdated="2023-07-17"
/>
export default function TermsOfService() {
const lastUpdated = "2023-07-17"; // Ideally, this should be fetched from a server or stored in a config file
return (
<>
<PrivacyTermsBanner
title={metadata.title as string}
lastUpdated={lastUpdated}
/>

@samanyougarg samanyougarg changed the title Add terms and privacy pages Add terms and privacy pages #178 Jul 16, 2023
@samanyougarg samanyougarg changed the title Add terms and privacy pages #178 Add terms and privacy pages #131 Jul 16, 2023
@vercel vercel bot temporarily deployed to Preview – bg-frontend July 16, 2023 19:40 Inactive
@samanyougarg samanyougarg merged commit 384ec57 into main Jul 16, 2023
9 checks passed
@samanyougarg samanyougarg deleted the add-terms branch July 16, 2023 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant