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

Feature/copy UUID link to clipboard #1394

Conversation

ridoo
Copy link

@ridoo ridoo commented Mar 21, 2023

A suggestion to let the user copy the UUID-Link of a resource, instead of the detail url. Along with the UUID-resolver (see geonode PR here) the target will resolve to the detail url anyways, but gives the user the UUID-Link at hand.

@ridoo ridoo force-pushed the feature/copy-uuid-link-to-clipboard branch from e277431 to 3b99bd2 Compare March 24, 2023 12:02
@ridoo ridoo changed the base branch from 4.1.x to master March 24, 2023 12:02
@ridoo ridoo force-pushed the feature/copy-uuid-link-to-clipboard branch 3 times, most recently from 3f57ab4 to 847480e Compare March 24, 2023 12:22
The UUID link provides the user a more generic permalink (no database id
involved) which redirects to the resource, anyway.

Requires UUID resolver at the GeoNode backend.
see GeoNode/geonode#10375
@ridoo ridoo force-pushed the feature/copy-uuid-link-to-clipboard branch from 847480e to 8245882 Compare March 24, 2023 12:24
@ridoo
Copy link
Author

ridoo commented Mar 24, 2023

So, finally resolved the messed up merge. Sorry for the confusion.

@giohappy
Copy link

@ridoo I like your proposal. I'll let @allyoucanmap review it.

Copy link
Collaborator

@allyoucanmap allyoucanmap left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @ridoo!

@allyoucanmap allyoucanmap merged commit 608599b into GeoNode:master Apr 11, 2023
@ridoo ridoo deleted the feature/copy-uuid-link-to-clipboard branch April 18, 2023 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants