Skip to content

Displaying UUIDs on asset details - #334

Merged
lucaspar merged 3 commits into
masterfrom
lp/uuids
Aug 20, 2026
Merged

Displaying UUIDs on asset details#334
lucaspar merged 3 commits into
masterfrom
lp/uuids

Conversation

@lucaspar

@lucaspar lucaspar commented Aug 20, 2026

Copy link
Copy Markdown
Member

SFDS-297

Changes to improve SDK usability from the website.

  • UUIDs to datasets and capture details;
  • Added relevant SDK docs links to multiple pages.

Note

Low Risk
Template and small JS wiring only; no auth, data, or API behavior changes. Risk is limited to copy-button wiring and extra external doc links.

Overview
Makes it easier to use the Python SDK from the website by showing copyable UUIDs and linking to SDK docs on the relevant screens.

Capture and dataset details modals now display the UUID with a Copy UUID control (dataset copy moved off the name row). Capture afterInject wires the existing attachUuidCopyButton helper. File detail does the same for the file UUID.

Info banners and footer links point to getting started, captures/datasets/files APIs, and upload/download workflows on files, API key, published datasets, and upload-capture pages.

Reviewed by Cursor Bugbot for commit aa2f335. Configure here.

@lucaspar lucaspar self-assigned this Aug 20, 2026
@lucaspar lucaspar added documentation Improvements or additions to documentation styling Special focus on styling of front-end components javascript Pull requests that update non-trivial javascript code labels Aug 20, 2026
@lucaspar
lucaspar requested a review from mjcurran August 20, 2026 15:53
@lucaspar
lucaspar marked this pull request as ready for review August 20, 2026 15:54

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit aa2f335. Configure here.

Comment thread gateway/sds_gateway/static/js/constants/detailsModalConfig.js
Comment thread gateway/sds_gateway/templates/users/file_detail.html

@mjcurran mjcurran left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tested locally, all looks good to me.
I was working on the UUID ticket as an exercise, so it's helpful to see where you made similar or different choices in implementing that part.

@lucaspar
lucaspar merged commit 0fee041 into master Aug 20, 2026
3 checks passed
@lucaspar
lucaspar deleted the lp/uuids branch August 20, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation javascript Pull requests that update non-trivial javascript code styling Special focus on styling of front-end components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants