Skip to content

Conversation

@abhinavohri
Copy link
Contributor

Resolves: #7712

Summary:
Added "Duplicate" action button in the three-dot menu for each appointment schedule.

Added a new isCreatingNew flag in AppointmentConfigListItem to control whether to edit or duplicate the existing config.

Before:
image

After:
image

@SebastianKrupinski
Copy link
Contributor

Tested works good.

@SebastianKrupinski
Copy link
Contributor

Hi @abhinavohri

Code looks good and I have no issue accepting it.

Only one change/addition, can we add a "(duplicate)" or "(copy)" to the title once the editor opens.

@abhinavohri
Copy link
Contributor Author

Hi @SebastianKrupinski , sure I can add that.

@SebastianKrupinski
Copy link
Contributor

Hi @abhinavohri

Thank you for the updated PR, but this is not exactly what I had in mind, by title, I meant the "Appointment Name", most users will not look at the modal title.

image

@abhinavohri
Copy link
Contributor Author

@SebastianKrupinski My apologies, I would fix it. Just to be sure the name should be 30 min meeting(duplicate).

@codecov
Copy link

codecov bot commented Jan 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@SebastianKrupinski
Copy link
Contributor

@abhinavohri

Tested your latest changes! Works good.

image

@SebastianKrupinski
Copy link
Contributor

@rakekniven

For translation is there a issue with having "(duplicate)" or should the brackets be separated?

@rakekniven
Copy link
Member

@SebastianKrupinski My apologies, I would fix it. Just to be sure the name should be 30 min meeting(duplicate).

I would prefer "30 min meeting (copy)."

@rakekniven

For translation is there a issue with having "(duplicate)" or should the brackets be separated?

No, not at all.

@abhinavohri
Copy link
Contributor Author

abhinavohri commented Jan 6, 2026

@rakekniven I have changed it to "(copy)"

Comment on lines 301 to 303
if (this.isDuplicate) {
this.editing.name = `${this.editing.name} (copy)`
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "(copy)" needs to be wrapped in the $t() so that it can be translated... t('calendar', '(copy)')

@SebastianKrupinski
Copy link
Contributor

@rakekniven thank you

@SebastianKrupinski
Copy link
Contributor

@abhinavohri

I am ready to merge these changes, can you squash the commits in to a single commit and rebase to the latest master.

Thank you

Signed-off-by: Abhinav Ohri <abhinavohri13@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Appointments: Clone a schedule

3 participants