Datenschutz und Impressum MD's hinzugefügt und calls angepasst.#162
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds legal Markdown documents (Datenschutzerklärung + Impressum) to the frontend and wires up navigation links to them from the homepage header.
Changes:
- Added
Datenschutz.mdwith a full privacy policy text. - Added
Impressum.mdwith imprint/contact information. - Replaced placeholder homepage links with anchors pointing to the new Markdown files.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
project/frontend/app/homepage/MarkdownFiles/Impressum.md |
New Impressum Markdown document. |
project/frontend/app/homepage/MarkdownFiles/Datenschutz.md |
New Datenschutzerklärung Markdown document. |
project/frontend/app/homepage/homepage.tsx |
Adds header links intended to reach the new legal documents. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| Im Rahmen unserer Dienste können wir Produkte verkaufen, die unbeabsichtigt über unsere Partner personenbezogene Daten aus besonderen Kategorien preisgeben, wie gesundheitsbezogene Informationen (z. B. Allergien oder Ernährungsbedürfnisse), Informationen über deine Religion (z. B. zur Einhaltung deiner religiösen Ernährungseinschränkungen), Informationen zu deinem gesundheitlichen Zustand (z. B. Apothekenprodukte, Medikamente, medizinische Geräte, medikamentöse Cremes, Nahrungsergänzungsmittel oder pflanzliche/homöopathische Produkte) oder über dein Sexualleben. Wir erheben und verarbeiten Gesundheitsdaten wie Medikamente mit deiner Zustimmung im Rahmen deiner Bestellung. | ||
|
|
||
| Wir auch beschränkte Produkten oder Dienste wie Alkohol, E-Zigaretten oder Tabak verkaufen und liefern. Zur Erfüllung unserer rechtlichen Verpflichtungen nach geltendem Recht müssen wir für den Verkauf und die Lieferung solcher Produkte möglicherweise Verifizierungsverfahren einsetzen, wie unter anderem zur Verifizierung deines Alters und deiner Identität, indem unsere Fahrer:innen deinen Identitätsnachweis überprüfen, ggf. automatisch per Scan deines Ausweises. Zu diesem Zweck benötigen wir möglicherweise als Beweis ein gültiges, staatlich ausgestelltes Ausweisdokument. Bitte beachte, dass Lazycook, falls du dich dem Verifizierungsprozess verweigerst, nicht verpflichtet ist, dir die Produkte oder Dienste bereitzustellen. Die rechtliche Grundlage für diese Verarbeitung personenbezogener Daten ist die Einhaltung der gesetzlichen Verpflichtung. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
…essum&Datenschutz
…th navigation links
…essum and Datenschutz pages
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (3)
project/frontend/app/components/markdown-renderer.tsx:43
- The
coderenderer likely receives non-DOM props fromreact-markdown(commonlyinline,node, and sometimesclassNamefor fenced code blocks). Spreading them onto<code>can produce invalid attributes and also prevents distinguishing inline vs block code. Consider destructuringinline/node(and handlingclassName/language) instead of blindly spreading.
code: (props: ComponentPropsWithoutRef<"code">) => (
<code className="rounded bg-gray-100 px-1.5 py-0.5 font-mono text-sm text-gray-800" {...props} />
),
project/frontend/app/datenschutz/page.tsx:28
- This page renders its own
<h1>and the markdown file also starts with a# ...heading, which will result in multiple H1s and duplicated titles (accessibility/SEO). Either remove the page-level heading and rely on the markdown heading, or adjust the renderer/markdown so the top-level markdown heading is demoted toh2.
<main className="min-h-screen bg-white px-6 py-12">
<div className="mx-auto max-w-4xl">
<h1 className="text-3xl font-semibold mb-6">Datenschutzerklärung</h1>
<p className="text-gray-600 mb-8">
Die vollständige Datenschutzerklärung wird hier aus der bestehenden Markdown-Datei angezeigt.
</p>
<MarkdownRenderer content={content} />
</div>
project/frontend/app/impressum/page.tsx:28
- This page renders its own
<h1>and the markdown file also starts with a# ...heading, which will result in multiple H1s and duplicated titles (accessibility/SEO). Either remove the page-level heading and rely on the markdown heading, or adjust the renderer/markdown so the top-level markdown heading is demoted toh2.
<main className="min-h-screen bg-white px-6 py-12">
<div className="mx-auto max-w-4xl">
<h1 className="text-3xl font-semibold mb-6">Impressum</h1>
<p className="text-gray-600 mb-8">
Das vollständige Impressum wird hier aus der bestehenden Markdown-Datei angezeigt.
</p>
<MarkdownRenderer content={content} />
</div>
| import type { ComponentPropsWithoutRef } from "react"; | ||
|
|
||
| type MarkdownRendererProps = { | ||
| content: string; | ||
| }; | ||
|
|
||
| const markdownComponents = { | ||
| h1: (props: ComponentPropsWithoutRef<"h1">) => ( | ||
| <h1 className="mb-6 text-3xl font-semibold text-gray-900" {...props} /> | ||
| ), | ||
| h2: (props: ComponentPropsWithoutRef<"h2">) => ( | ||
| <h2 className="mb-4 mt-8 text-2xl font-semibold text-gray-900" {...props} /> | ||
| ), | ||
| h3: (props: ComponentPropsWithoutRef<"h3">) => ( | ||
| <h3 className="mb-3 mt-6 text-xl font-semibold text-gray-900" {...props} /> | ||
| ), | ||
| p: (props: ComponentPropsWithoutRef<"p">) => ( | ||
| <p className="mb-4 leading-7 text-gray-700" {...props} /> | ||
| ), | ||
| ul: (props: ComponentPropsWithoutRef<"ul">) => ( | ||
| <ul className="mb-4 ml-6 list-disc space-y-2 text-gray-700" {...props} /> | ||
| ), | ||
| ol: (props: ComponentPropsWithoutRef<"ol">) => ( | ||
| <ol className="mb-4 ml-6 list-decimal space-y-2 text-gray-700" {...props} /> | ||
| ), | ||
| li: (props: ComponentPropsWithoutRef<"li">) => ( | ||
| <li className="leading-7" {...props} /> | ||
| ), | ||
| strong: (props: ComponentPropsWithoutRef<"strong">) => ( | ||
| <strong className="font-semibold text-gray-900" {...props} /> | ||
| ), | ||
| a: (props: ComponentPropsWithoutRef<"a">) => ( | ||
| <a className="text-blue-600 underline underline-offset-2 hover:text-blue-700" {...props} /> | ||
| ), | ||
| blockquote: (props: ComponentPropsWithoutRef<"blockquote">) => ( | ||
| <blockquote className="mb-4 border-l-4 border-gray-300 pl-4 italic text-gray-600" {...props} /> | ||
| ), | ||
| hr: () => <hr className="my-8 border-gray-200" />, | ||
| code: (props: ComponentPropsWithoutRef<"code">) => ( |
| function readMarkdownFile(fileName: string) { | ||
| try { | ||
| return readFileSync( | ||
| path.join(process.cwd(), "app", "homepage", "MarkdownFiles", fileName), | ||
| "utf8" | ||
| ); | ||
| } catch { | ||
| notFound(); | ||
| } | ||
| } |
There was a problem hiding this comment.
Implemented in 9a07002: extracted the duplicated readMarkdownFile into app/lib/read-markdown.ts and switched /datenschutz and /impressum pages to use the shared helper.
| function readMarkdownFile(fileName: string) { | ||
| try { | ||
| return readFileSync( | ||
| path.join(process.cwd(), "app", "homepage", "MarkdownFiles", fileName), | ||
| "utf8" | ||
| ); | ||
| } catch { | ||
| notFound(); | ||
| } | ||
| } |
|
|
||
| Im Rahmen unserer Dienste können wir Produkte verkaufen, die unbeabsichtigt über unsere Partner personenbezogene Daten aus besonderen Kategorien preisgeben, wie gesundheitsbezogene Informationen (z. B. Allergien oder Ernährungsbedürfnisse), Informationen über deine Religion (z. B. zur Einhaltung deiner religiösen Ernährungseinschränkungen), Informationen zu deinem gesundheitlichen Zustand (z. B. Apothekenprodukte, Medikamente, medizinische Geräte, medikamentöse Cremes, Nahrungsergänzungsmittel oder pflanzliche/homöopathische Produkte) oder über dein Sexualleben. Wir erheben und verarbeiten Gesundheitsdaten wie Medikamente mit deiner Zustimmung im Rahmen deiner Bestellung. | ||
|
|
||
| Wir auch beschränkte Produkten oder Dienste wie Alkohol, E-Zigaretten oder Tabak verkaufen und liefern. Zur Erfüllung unserer rechtlichen Verpflichtungen nach geltendem Recht müssen wir für den Verkauf und die Lieferung solcher Produkte möglicherweise Verifizierungsverfahren einsetzen, wie unter anderem zur Verifizierung deines Alters und deiner Identität, indem unsere Fahrer:innen deinen Identitätsnachweis überprüfen, ggf. automatisch per Scan deines Ausweises. Zu diesem Zweck benötigen wir möglicherweise als Beweis ein gültiges, staatlich ausgestelltes Ausweisdokument. Bitte beachte, dass Lazycook, falls du dich dem Verifizierungsprozess verweigerst, nicht verpflichtet ist, dir die Produkte oder Dienste bereitzustellen. Die rechtliche Grundlage für diese Verarbeitung personenbezogener Daten ist die Einhaltung der gesetzlichen Verpflichtung. |
Agent-Logs-Url: https://github.com/GalacticCodeGambit/LazyCook/sessions/9460d714-3139-45d5-b2e2-e6ee6e3d8563 Co-authored-by: GalacticCodeGambit <150372421+GalacticCodeGambit@users.noreply.github.com>
* Added Algorithm, which returns a sorted list of recipes of an matching Ingridients List * Added Filtering * Tests für Database, passwort überprüfung und email überürufung wurden geschrieben. Die dazugehörigen Abhängigkeiten in requirements.txt angepasst. Sie werden in test_main.py aufgerufen. Namens faktorisierung ist schwer bei den Klassennamen da bei tests der pycharm Debugger eher auf klassen mit test_ vorne guckt. Deswegen leider test_ * ci pipeline abhägigkeit mit rein gemacht * Feat bei Ci yml entfernt * Delete project/data/LazyCookDB.sqlite3 * Feature: Forgot Password link in signin modal. Popup for Email Input. Konto deletion reactivation. * Feat/ci cd add lint (#142) * Rename models.py to Models.py * Rename auth.py to Auth.py * Rename Routes.py to routes.py * Added Refactoring Summary * Refactor documentation for code naming conventions * Fix spelling errors in refactoring documentation Corrected spelling of 'Refaktorisierung' and 'Frontend'. * Delete project/data/LazyCookDB.sqlite3 * AI Agent md's changed * Passwort und Email Änderung zu Popups geändert. und neugeordnet untereinander * Update README.md to mark tasks as completed for account management features * Add CI/CD pipeline and Super Linter configuration * Test für lint * Test ci/cd lint.yml * Test ci/cd lint.yml * Test ci/cd lint.yml * Test ci/cd lint.yml regex Filter * Test ci/cd lint.yml * Test ci/cd lint.yml * Test ci/cd lint.yml regex Filter angepasst * Tests für Database, passwort überprüfung und email überürufung wurden geschrieben. Die dazugehörigen Abhängigkeiten in requirements.txt angepasst. Sie werden in test_main.py aufgerufen. Namens faktorisierung ist schwer bei den Klassennamen da bei tests der pycharm Debugger eher auf klassen mit test_ vorne guckt. Deswegen leider test_ * ci pipeline abhägigkeit mit rein gemacht * Feat bei Ci yml entfernt * Update FILTER_REGEX_INCLUDE pattern in lint.yml * Update lint.yml * Enable validation for the entire codebase * Update lint.yml * Update lint.yml * Enable validation for all codebase in lint workflow * Remove Python Black and Markdown validation * Update lint workflow to include additional validations * Update FILTER_REGEX_INCLUDE to support YAML files * Update lint.yml * Update lint.yml * Refactor CI configuration to use Docker Compose v2 syntax and add GitHub Super Linter step * Test ci.yml * Update ci.yml * Enable Flake8 validation for Python files * Test ci.yml * Remove 'feat/*' branch from push trigger in CI configuration * Python black gelöst --------- Co-authored-by: Samuel Goebel <goebelsamuel@gmail.com> Co-authored-by: Eden Tabea Bernhard <105359952+EdenBernhard@users.noreply.github.com> Co-authored-by: Nicoolaus <162422307+Nicoolaus@users.noreply.github.com> Co-authored-by: Hellocrafting <alexanderfgroer@gmail.com> Co-authored-by: nicla <niclas.matzke@gmail.com> Co-authored-by: Hellocrafting <75727565+Hellocrafting@users.noreply.github.com> * update super-linter to version 8.6.0 in CI configuration * Test Linter * Test Linter * Feature: Email and Password change popup width changed * Separate linter configuration into its own workflow file (#149) * Separate linter configuration into its own workflow file Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Update README.md to reflect linter workflow separation and installation instructions --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Update Node.js version in Dockerfile-frontend to 24.1.0 * Tested the recipe algorithim * Feature: Fix problem with refresh token and include show Password * Feature: forgot password functionality works with redirection to homepage * added SaveInDB Methods * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Fix: Copilot review suggestion * Fix: Copilot review suggestion * Fix: Copilot review suggestion * Fix: Copilot review suggestion * Feat: forgot password functionality (#151) * Feature: Forgot Password link in signin modal. Popup for Email Input. Konto deletion reactivation. * Feature: Email and Password change popup width changed * Feature: Fix problem with refresh token and include show Password * Feature: forgot password functionality works with redirection to homepage * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Fix: Copilot review suggestion * fix: make password visibility toggle keyboard focusable Agent-Logs-Url: https://github.com/GalacticCodeGambit/LazyCook/sessions/bf26f4ce-caa3-43fa-b940-13d7be23d7bf Co-authored-by: EdenBernhard <105359952+EdenBernhard@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * Feat: show top 5 incredients as options (#152) * Feature: Forgot Password link in signin modal. Popup for Email Input. Konto deletion reactivation. * Feature: Email and Password change popup width changed * Feature: Fix problem with refresh token and include show Password * Feature: forgot password functionality works with redirection to homepage * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Fix: Copilot review suggestion * Fix: Copilot review suggestion * Fix: Copilot review suggestion * Fix: Copilot review suggestion * Feat: Ingredients suggestions. Top 5 most typed shown in popup * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Feat: Ingredients suggestions. Top 5 most typed shown in popup --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Black used * Info: Standort von Impressum und Datenschutz * Added Review-Protokoll-01 * Clear participant roles in review protocol Removed participant names from the review protocol. * Update participants and review focus in documentation * Made the required RecipeSUCUK changes * Rename RecipeSucuk.py to RecipeSUCUK.py * Added Search Algorithm for Recipe Name search * SearchRecipeNames now returns a list of recipe Objects * Datenschutz und Impressum MD's hinzugefügt und calls angepasst. (#162) * Datenschutz und Impressum MD's hinzugefügt und calls angepasst. * Datenschutz und Impressum MD's hinzugefügt und calls angepasst. (Kommentar war invalide) * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Update navigation comments for Datenschutz and Impressum links * Fix Datenschutz and Impressum, Add Datenschutz and Impressum pages with navigation links * Add MarkdownRenderer component for rendering Markdown content in Impressum and Datenschutz pages * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * refactor: extract shared markdown reader helper Agent-Logs-Url: https://github.com/GalacticCodeGambit/LazyCook/sessions/9460d714-3139-45d5-b2e2-e6ee6e3d8563 Co-authored-by: GalacticCodeGambit <150372421+GalacticCodeGambit@users.noreply.github.com> * Update Impressum.md --------- Co-authored-by: F <150372421+GalacticCodeGambit@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> --------- Co-authored-by: Samuel Goebel <goebelsamuel@gmail.com> Co-authored-by: nicla <niclas.matzke@gmail.com> Co-authored-by: Eden Bernhard <etbernhard4@gmail.com> Co-authored-by: Eden Tabea Bernhard <105359952+EdenBernhard@users.noreply.github.com> Co-authored-by: Nicoolaus <162422307+Nicoolaus@users.noreply.github.com> Co-authored-by: Hellocrafting <alexanderfgroer@gmail.com> Co-authored-by: Hellocrafting <75727565+Hellocrafting@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>



No description provided.