From 136883fd94872322bb2334e66f119f6c9488b8e0 Mon Sep 17 00:00:00 2001 From: Philipp Heckel Date: Sun, 10 Apr 2022 15:13:12 -0400 Subject: [PATCH] Additional descriptions for settings (#203), URL validation (#204) --- docs/releases.md | 3 +- web/public/static/langs/en.json | 17 ++++- web/src/app/utils.js | 41 ++++++++--- web/src/components/Notifications.js | 4 +- web/src/components/Preferences.js | 104 +++++++++++++++++++--------- 5 files changed, 123 insertions(+), 46 deletions(-) diff --git a/docs/releases.md b/docs/releases.md index 85b33a1..91d32df 100644 --- a/docs/releases.md +++ b/docs/releases.md @@ -24,8 +24,9 @@ and the [ntfy Android app](https://github.com/binwiederhier/ntfy-android/release **Bugs:** -* Web app: English language strings fixes ([#203](https://github.com/binwiederhier/ntfy/issues/203), thanks to [@StoyanDimitrov](https://github.com/StoyanDimitrov)) +* Web app: English language strings fixes, additional descriptions for settings ([#203](https://github.com/binwiederhier/ntfy/issues/203), thanks to [@StoyanDimitrov](https://github.com/StoyanDimitrov)) * Web app: Show error message snackbar when sending test notification fails ([#205](https://github.com/binwiederhier/ntfy/issues/205), thanks to [@cmeis](https://github.com/cmeis)) +* Web app: basic URL validation in user management ([#204](https://github.com/binwiederhier/ntfy/issues/204), thanks to [@cmeis](https://github.com/cmeis)) **Translations (web app):** diff --git a/web/public/static/langs/en.json b/web/public/static/langs/en.json index a2dca26..a27ff86 100644 --- a/web/public/static/langs/en.json +++ b/web/public/static/langs/en.json @@ -33,7 +33,7 @@ "notifications_none_for_any_title": "You haven't received any notifications.", "notifications_none_for_any_description": "To send notifications to a topic, simply PUT or POST to the topic URL. Here's an example using one of your topics.", "notifications_no_subscriptions_title": "It looks like you don't have any subscriptions yet.", - "notifications_no_subscriptions_description": "Click the \"Add subscription\" link to create or subscribe to a topic. After that, you can send messages via PUT or POST and you'll receive notifications here.", + "notifications_no_subscriptions_description": "Click the \"{{linktext}}\" link to create or subscribe to a topic. After that, you can send messages via PUT or POST and you'll receive notifications here.", "notifications_example": "Example", "notifications_more_details": "For more information, check out the website or documentation.", "notifications_loading": "Loading notifications …", @@ -103,8 +103,13 @@ "subscribe_dialog_error_user_anonymous": "anonymous", "prefs_notifications_title": "Notifications", "prefs_notifications_sound_title": "Notification sound", + "prefs_notifications_sound_description_none": "Notifications do not play any sound when they arrive", + "prefs_notifications_sound_description_some": "Notifications play the {{sound}} sound when they arrive", "prefs_notifications_sound_no_sound": "No sound", "prefs_notifications_min_priority_title": "Minimum priority", + "prefs_notifications_min_priority_description_any": "Showing all notifications, regardless of priority", + "prefs_notifications_min_priority_description_x_or_higher": "Show notifications if priority is {{number}} ({{name}}) or above", + "prefs_notifications_min_priority_description_max": "Show notifications if priority is 5 (max)", "prefs_notifications_min_priority_any": "Any priority", "prefs_notifications_min_priority_low_and_higher": "Low priority and higher", "prefs_notifications_min_priority_default_and_higher": "Default priority and higher", @@ -116,6 +121,11 @@ "prefs_notifications_delete_after_one_day": "After one day", "prefs_notifications_delete_after_one_week": "After one week", "prefs_notifications_delete_after_one_month": "After one month", + "prefs_notifications_delete_after_never_description": "Notifications are never auto-deleted", + "prefs_notifications_delete_after_three_hours_description": "Notifications are auto-deleted after three hours", + "prefs_notifications_delete_after_one_day_description": "Notifications are auto-deleted after one day", + "prefs_notifications_delete_after_one_week_description": "Notifications are auto-deleted after one week", + "prefs_notifications_delete_after_one_month_description": "Notifications are auto-deleted after one month", "prefs_users_title": "Manage users", "prefs_users_description": "Add/remove users for your protected topics here. Please note that username and password are stored in the browser's local storage.", "prefs_users_add_button": "Add user", @@ -131,6 +141,11 @@ "prefs_users_dialog_button_save": "Save", "prefs_appearance_title": "Appearance", "prefs_appearance_language_title": "Language", + "priority_min": "min", + "priority_low": "low", + "priority_default": "default", + "priority_high": "high", + "priority_max": "max", "error_boundary_title": "Oh no, ntfy crashed", "error_boundary_description": "This should obviously not happen. Very sorry about this.
If you have a minute, please report this on GitHub, or let us know via Discord or Matrix.", "error_boundary_button_copy_stack_trace": "Copy stack trace", diff --git a/web/src/app/utils.js b/web/src/app/utils.js index 2f04dfd..adba2e9 100644 --- a/web/src/app/utils.js +++ b/web/src/app/utils.js @@ -24,7 +24,7 @@ export const expandUrl = (url) => [`https://${url}`, `http://${url}`]; export const expandSecureUrl = (url) => `https://${url}`; export const validUrl = (url) => { - return url.match(/^https?:\/\//); + return url.match(/^https?:\/\/.+/); } export const validTopic = (topic) => { @@ -153,17 +153,38 @@ export const openUrl = (url) => { }; export const sounds = { - "beep": beep, - "juntos": juntos, - "pristine": pristine, - "ding": ding, - "dadum": dadum, - "pop": pop, - "pop-swoosh": popSwoosh + "ding": { + file: ding, + label: "Ding" + }, + "juntos": { + file: juntos, + label: "Juntos" + }, + "pristine": { + file: pristine, + label: "Pristine" + }, + "dadum": { + file: dadum, + label: "Dadum" + }, + "pop": { + file: pop, + label: "Pop" + }, + "pop-swoosh": { + file: popSwoosh, + label: "Pop swoosh" + }, + "beep": { + file: beep, + label: "Beep" + } }; -export const playSound = async (sound) => { - const audio = new Audio(sounds[sound]); +export const playSound = async (id) => { + const audio = new Audio(sounds[id].file); return audio.play(); }; diff --git a/web/src/components/Notifications.js b/web/src/components/Notifications.js index eb33880..a412e29 100644 --- a/web/src/components/Notifications.js +++ b/web/src/components/Notifications.js @@ -389,7 +389,9 @@ const NoSubscriptions = () => { {t("notifications_no_subscriptions_title")} - {t("notifications_no_subscriptions_description")} + {t("notifications_no_subscriptions_description", { + linktext: t("nav_button_subscribe") + })} diff --git a/web/src/components/Preferences.js b/web/src/components/Preferences.js index e6f2819..11b688e 100644 --- a/web/src/components/Preferences.js +++ b/web/src/components/Preferences.js @@ -32,8 +32,13 @@ import DialogTitle from "@mui/material/DialogTitle"; import DialogContent from "@mui/material/DialogContent"; import DialogActions from "@mui/material/DialogActions"; import userManager from "../app/UserManager"; -import {playSound, shuffle} from "../app/utils"; +import {playSound, shuffle, sounds, validUrl} from "../app/utils"; import {useTranslation} from "react-i18next"; +import priority1 from "../img/priority-1.svg"; +import priority2 from "../img/priority-2.svg"; +import priority3 from "../img/priority-3.svg"; +import priority4 from "../img/priority-4.svg"; +import priority5 from "../img/priority-5.svg"; const Preferences = () => { return ( @@ -51,7 +56,7 @@ const Notifications = () => { const { t } = useTranslation(); return ( - + {t("prefs_notifications_title")} @@ -72,19 +77,19 @@ const Sound = () => { if (!sound) { return null; // While loading } + let description; + if (sound === "none") { + description = t("prefs_notifications_sound_description_none"); + } else { + description = t("prefs_notifications_sound_description_some", { sound: sounds[sound].label }); + } return ( - +
playSound(sound)} disabled={sound === "none"}> @@ -104,8 +109,26 @@ const MinPriority = () => { if (!minPriority) { return null; // While loading } + const priorities = { + 1: t("priority_min"), + 2: t("priority_low"), + 3: t("priority_default"), + 4: t("priority_high"), + 5: t("priority_max") + } + let description; + if (minPriority === 1) { + description = t("prefs_notifications_min_priority_description_any"); + } else if (minPriority === 5) { + description = t("prefs_notifications_min_priority_description_max"); + } else { + description = t("prefs_notifications_min_priority_description_x_or_higher", { + number: minPriority, + name: priorities[minPriority] + }); + } return ( - + {t("prefs_notifications_delete_after_never")} @@ -145,10 +177,7 @@ const DeleteAfter = () => { const PrefGroup = (props) => { return ( -
+
{props.children}
) @@ -156,26 +185,31 @@ const PrefGroup = (props) => { const Pref = (props) => { return ( - <> +
- {props.title} +
{props.title}
+ {props.description &&
{props.description}
}
{props.children}
- +
); }; @@ -202,8 +236,8 @@ const Users = () => { }; return ( - - + + {t("prefs_users_title")} @@ -260,7 +294,7 @@ const UserTable = (props) => { - {t("prefs_users_table_user_header")} + {t("prefs_users_table_user_header")} {t("prefs_users_table_base_url_header")} @@ -271,7 +305,7 @@ const UserTable = (props) => { key={user.baseUrl} sx={{ '&:last-child td, &:last-child th': { border: 0 } }} > - {user.username} + {user.username} {user.baseUrl} handleEditClick(user)}> @@ -307,8 +341,12 @@ const UserDialog = (props) => { if (editMode) { return username.length > 0 && password.length > 0; } + const baseUrlValid = validUrl(baseUrl); const baseUrlExists = props.users?.map(user => user.baseUrl).includes(baseUrl); - return !baseUrlExists && username.length > 0 && password.length > 0; + return baseUrlValid + && !baseUrlExists + && username.length > 0 + && password.length > 0; })(); const handleSubmit = async () => { props.onSubmit({ @@ -373,7 +411,7 @@ const Appearance = () => { const { t } = useTranslation(); return ( - + {t("prefs_appearance_title")}