refactor(notifications): use "Daily Reminder" terminology consistently
- Rename "Reminder Notification(s)" to "Daily Reminder" in Account and Help views
- Update NOTIFY_PUSH_SUCCESS title/message ("Notifications On", "Daily Reminder notifications are now enabled.")
- Align plugin spec doc with "Notifications" section naming
This commit is contained in:
@@ -95,7 +95,7 @@ The rollover interval must be **stored with the schedule** in the plugin’s per
|
|||||||
|
|
||||||
## App-side behavior (for context)
|
## App-side behavior (for context)
|
||||||
|
|
||||||
- The app will only pass `rolloverIntervalMinutes` when a **dev-only** setting is enabled (e.g. “Use 10-minute rollover for testing” in the Reminder Notifications section). Production users will not set it.
|
- The app will only pass `rolloverIntervalMinutes` when a **dev-only** setting is enabled (e.g. “Use 10-minute rollover for testing” in the Notifications section). Production users will not set it.
|
||||||
- The app will pass it on every `scheduleDailyNotification` call when the user has that setting on (first-time enable and edit). When the user turns the setting off, the app will call `scheduleDailyNotification` without the parameter (so the plugin can persist “no interval” / 24h).
|
- The app will pass it on every `scheduleDailyNotification` call when the user has that setting on (first-time enable and edit). When the user turns the setting off, the app will call `scheduleDailyNotification` without the parameter (so the plugin can persist “no interval” / 24h).
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|||||||
@@ -67,7 +67,7 @@
|
|||||||
class="block w-full text-center text-md font-bold uppercase bg-blue-600 text-white mt-2 px-2 py-2 rounded-md"
|
class="block w-full text-center text-md font-bold uppercase bg-blue-600 text-white mt-2 px-2 py-2 rounded-md"
|
||||||
@click="handleTurnOnNotifications"
|
@click="handleTurnOnNotifications"
|
||||||
>
|
>
|
||||||
Turn on Daily Message
|
Turn on Daily Reminder
|
||||||
</button>
|
</button>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
|||||||
@@ -1642,9 +1642,8 @@ export const NOTIFY_PUSH_SETUP_UNDERWAY = {
|
|||||||
|
|
||||||
// Used in: PushNotificationPermission.vue (turnOnNotifications method - success)
|
// Used in: PushNotificationPermission.vue (turnOnNotifications method - success)
|
||||||
export const NOTIFY_PUSH_SUCCESS = {
|
export const NOTIFY_PUSH_SUCCESS = {
|
||||||
title: "Notification Is On",
|
title: "Notifications On",
|
||||||
message:
|
message: "Daily Reminder notifications are now enabled.",
|
||||||
"You should see at least one on your device; if not, check the 'Troubleshoot' link.",
|
|
||||||
};
|
};
|
||||||
|
|
||||||
// Used in: PushNotificationPermission.vue (turnOnNotifications method - general error)
|
// Used in: PushNotificationPermission.vue (turnOnNotifications method - general error)
|
||||||
|
|||||||
@@ -86,10 +86,10 @@
|
|||||||
<h2 id="notificationsHeading" class="mb-2 font-bold">Notifications</h2>
|
<h2 id="notificationsHeading" class="mb-2 font-bold">Notifications</h2>
|
||||||
<div class="flex items-center justify-between mb-2">
|
<div class="flex items-center justify-between mb-2">
|
||||||
<div>
|
<div>
|
||||||
Reminder Notification
|
Daily Reminder
|
||||||
<button
|
<button
|
||||||
class="text-slate-400 fa-fw cursor-pointer"
|
class="text-slate-400 fa-fw cursor-pointer"
|
||||||
aria-label="Learn more about reminder notifications"
|
aria-label="Learn more about Daily Reminder notifications"
|
||||||
@click.stop="showReminderNotificationInfo"
|
@click.stop="showReminderNotificationInfo"
|
||||||
>
|
>
|
||||||
<font-awesome
|
<font-awesome
|
||||||
@@ -102,7 +102,7 @@
|
|||||||
class="relative ml-2 cursor-pointer"
|
class="relative ml-2 cursor-pointer"
|
||||||
role="switch"
|
role="switch"
|
||||||
:aria-checked="notifyingReminder"
|
:aria-checked="notifyingReminder"
|
||||||
aria-label="Toggle reminder notifications"
|
aria-label="Toggle Daily Reminder notifications"
|
||||||
tabindex="0"
|
tabindex="0"
|
||||||
@click.stop.prevent="showReminderNotificationChoice()"
|
@click.stop.prevent="showReminderNotificationChoice()"
|
||||||
>
|
>
|
||||||
@@ -1257,7 +1257,7 @@ export default class AccountViewView extends Vue {
|
|||||||
if (this.isNativePlatform) {
|
if (this.isNativePlatform) {
|
||||||
// Show confirmation dialog
|
// Show confirmation dialog
|
||||||
this.notify.confirm(
|
this.notify.confirm(
|
||||||
"Would you like to turn off reminder notifications?",
|
"Would you like to turn off Daily Reminder notifications?",
|
||||||
async () => {
|
async () => {
|
||||||
try {
|
try {
|
||||||
// Cancel the native notification
|
// Cancel the native notification
|
||||||
@@ -1274,7 +1274,7 @@ export default class AccountViewView extends Vue {
|
|||||||
this.notifyingReminderTime = "";
|
this.notifyingReminderTime = "";
|
||||||
|
|
||||||
this.notify.success(
|
this.notify.success(
|
||||||
"Reminder notifications have been turned off.",
|
"Daily Reminder notifications have been turned off.",
|
||||||
TIMEOUTS.STANDARD,
|
TIMEOUTS.STANDARD,
|
||||||
);
|
);
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
|
|||||||
@@ -30,11 +30,11 @@
|
|||||||
<div>
|
<div>
|
||||||
<p>There are two types of notifications:</p>
|
<p>There are two types of notifications:</p>
|
||||||
|
|
||||||
<h2 class="text-xl font-semibold mt-4">Reminder Notifications</h2>
|
<h2 class="text-xl font-semibold mt-4">Daily Reminder Notifications</h2>
|
||||||
<div>
|
<div>
|
||||||
<p>
|
<p>
|
||||||
The Reminder Notification will be sent to you daily with a specific message,
|
The Daily Reminder notification will be sent to you daily with a specific
|
||||||
at whatever time you choose. Use it to remind
|
message, at whatever time you choose. Use it to remind
|
||||||
yourself to act, for example: pause and consider who has given you
|
yourself to act, for example: pause and consider who has given you
|
||||||
something, so you can record thanks in here.
|
something, so you can record thanks in here.
|
||||||
</p>
|
</p>
|
||||||
|
|||||||
Reference in New Issue
Block a user