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

"Appointment schedule" notion instead of only "Appointments" for a better UX #6371

Merged
merged 1 commit into from
Sep 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/components/AppNavigation/AppointmentConfigList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,15 @@
<div v-if="hasAtLeastOneCalendar"
class="appointment-config-list">
<AppNavigationCaption class="appointment-config-list__caption"
:name="t('calendar', 'Appointments')">
:name="t('calendar', 'Appointment schedules')">
<template v-if="hasUserEmailAddress"
#actions>
<ActionButton :close-after-click="true"
@click="showModalForNewConfig = true">
<template #icon>
<PlusIcon :size="20" decorative />
</template>
{{ t('calendar', 'Add new') }}
{{ t('calendar', 'Create new') }}
</ActionButton>
</template>
</AppNavigationCaption>
Expand Down
4 changes: 2 additions & 2 deletions src/components/AppointmentConfigModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -199,10 +199,10 @@
...mapStores(useAppointmentConfigsStore, useCalendarsStore, useSettingsStore),
formTitle() {
if (this.isNew) {
return this.$t('calendar', 'Create appointment')
return this.$t('calendar', 'Create appointment schedule')
}

return this.$t('calendar', 'Edit appointment')
return this.$t('calendar', 'Edit appointment schedule')

Check warning on line 205 in src/components/AppointmentConfigModal.vue

View check run for this annotation

Codecov / codecov/patch

src/components/AppointmentConfigModal.vue#L205

Added line #L205 was not covered by tests
},
saveButtonText() {
if (this.isNew) {
Expand Down
4 changes: 2 additions & 2 deletions src/components/AppointmentConfigModal/Confirmation.vue
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,10 @@
computed: {
title() {
if (this.isNew) {
return this.$t('calendar', 'Appointment was created successfully')
return this.$t('calendar', 'Appointment schedule successfully created')

Check warning on line 52 in src/components/AppointmentConfigModal/Confirmation.vue

View check run for this annotation

Codecov / codecov/patch

src/components/AppointmentConfigModal/Confirmation.vue#L52

Added line #L52 was not covered by tests
}

return this.$t('calendar', 'Appointment was updated successfully')
return this.$t('calendar', 'Appointment schedule successfully updated')
},
showCopyLinkButton() {
return navigator && navigator.clipboard
Expand Down
Loading