public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Shan Shaji <s.shaji@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH pve_flutter_frontend 2/3] ui: settings: add privacy policy url link in settings screen
Date: Fri, 29 Aug 2025 13:48:44 +0200	[thread overview]
Message-ID: <20250829114846.88507-3-s.shaji@proxmox.com> (raw)
In-Reply-To: <20250829114846.88507-1-s.shaji@proxmox.com>

According to Apple's App Store review guidelines all apps must include a
link to their privacy policy within the App [0]. To fix the issue add a
new list item in the settings screen that will allow users to access the
privacy policy.

[0] - https://developer.apple.com/app-store/review/guidelines/#legal

Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
---
 lib/pages/pve_settings_page.dart | 27 +++++++++++++++++++++++++++
 lib/utils/links.dart             |  2 ++
 2 files changed, 29 insertions(+)

diff --git a/lib/pages/pve_settings_page.dart b/lib/pages/pve_settings_page.dart
index 0d405c1..1f57f1c 100644
--- a/lib/pages/pve_settings_page.dart
+++ b/lib/pages/pve_settings_page.dart
@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:proxmox_login_manager/proxmox_general_settings_form.dart';
+import 'package:pve_flutter_frontend/utils/links.dart';
 import 'package:pve_flutter_frontend/widgets/pve_app_bar.dart';
 
 class PveSettingsPage extends StatelessWidget {
@@ -15,6 +16,32 @@ class PveSettingsPage extends StatelessWidget {
         child: Column(
           children: [
             ProxmoxGeneralSettingsForm(),
+            const ListTile(
+              title: Text(
+                'LEGAL',
+                style: TextStyle(
+                  fontWeight: FontWeight.bold,
+                ),
+              ),
+            ),
+            ListTile(
+              leading: const Icon(Icons.privacy_tip_outlined),
+              title: const Text('Privacy Policy'),
+              trailing: const Icon(Icons.open_in_new),
+              onTap: () {
+                try {
+                  tryLaunchUrl(Links.privacyPolicyUrl);
+                } catch (_) {
+                  ScaffoldMessenger.of(context).showSnackBar(
+                    const SnackBar(
+                      content: Text(
+                        'Could not launch privacy policy',
+                      ),
+                    ),
+                  );
+                }
+              },
+            ),
           ],
         ),
       ),
diff --git a/lib/utils/links.dart b/lib/utils/links.dart
index 29b0501..bab1b80 100644
--- a/lib/utils/links.dart
+++ b/lib/utils/links.dart
@@ -16,4 +16,6 @@ class Links {
       Uri.parse('https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-user');
   static final opaqueApp = Uri.parse(
       'https://play.google.com/store/apps/details?id=com.undatech.opaque');
+  static final privacyPolicyUrl = Uri.parse(
+      'https://pve.proxmox.com/wiki/Proxmox_VE_Mobile_Companion_Data_Protection');
 }
-- 
2.47.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


  parent reply	other threads:[~2025-08-29 11:49 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-29 11:48 [pve-devel] [PATCH proxmox_login_manager/pve_flutter_frontend 0/4] refactor settings screen and add HELP, LEGAL section in settings Shan Shaji
2025-08-29 11:48 ` [pve-devel] [PATCH pve_flutter_frontend 1/3] refactor: ui: add new settings page and use existing ssl toggle widget Shan Shaji
2025-08-29 11:48 ` Shan Shaji [this message]
2025-08-29 11:48 ` [pve-devel] [PATCH pve_flutter_frontend 3/3] ui: settings: add proxmox forum and bugzilla links Shan Shaji
2025-08-29 11:48 ` [pve-devel] [PATCH proxmox_login_manager 1/1] refactor: ui: move settings page to `pve_flutter_frontend` Shan Shaji

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250829114846.88507-3-s.shaji@proxmox.com \
    --to=s.shaji@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal