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

FTM: New UVE Toolbar - Implement "Copy URL" button #30732

Closed
Tracked by #30598
KevinDavilaDotCMS opened this issue Nov 21, 2024 · 2 comments · Fixed by #30819
Closed
Tracked by #30598

FTM: New UVE Toolbar - Implement "Copy URL" button #30732

KevinDavilaDotCMS opened this issue Nov 21, 2024 · 2 comments · Fixed by #30819

Comments

@KevinDavilaDotCMS
Copy link
Contributor

KevinDavilaDotCMS commented Nov 21, 2024

Parent Issue

#30598

Task

Implement the logic for the "Copy URL" button in the new UVE Toolbar component. This task involves modifying (if necessary) store logic for the newWithEditorToolbar() feature store, reusing existing copy URL functionality where possible.

Proposed Objective

Add the "Copy URL" button logic to the new toolbar and ensure it functions correctly under the feature flag, maintaining consistency with the existing copy URL functionality.

Acceptance Criteria

  • Ensure the "Copy URL" button functions correctly and integrates seamlessly with the new toolbar, in traditional and headless pages
  • The behaviour of the current "Copy URL" button should not be affected
Copy link

github-actions bot commented Dec 2, 2024

@zJaaal
Copy link
Contributor

zJaaal commented Dec 3, 2024

IQA Passed

It's working as expected, the Cannot GET /index error is related to the dev proxy we use. So it's not an issue.

DotCMS Docker Image: [dotcms/dotcms:trunk_a343172]

Screenshot

Screen.Recording.2024-12-03.at.12.43.01.PM-1.mov

@zJaaal zJaaal closed this as completed Dec 3, 2024
@zJaaal zJaaal moved this from Internal QA to QA - Backlog in dotCMS - Product Planning Dec 3, 2024
@nollymar nollymar moved this from QA - Backlog to Done in dotCMS - Product Planning Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants