-
Notifications
You must be signed in to change notification settings - Fork 90
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
Add buttons for copying canonical package URIs #1990
Conversation
@akarve pls wordsmith |
Codecov Report
@@ Coverage Diff @@
## master #1990 +/- ##
==========================================
- Coverage 46.80% 46.77% -0.03%
==========================================
Files 332 332
Lines 16418 16428 +10
Branches 2209 2187 -22
==========================================
Hits 7684 7684
- Misses 7837 7846 +9
- Partials 897 898 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code is fine. But I expected different functionality: copying immutable URI
@@ -84,6 +88,15 @@ function RevisionInfo({ revisionData, revision, bucket, name, path }) { | |||
_: R.identity, | |||
}) | |||
|
|||
const getHttpsUri = (r) => | |||
`${window.origin}${urls.bucketPackageTree(bucket, name, r.hash, path)}` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm... I expected quilt+s3://
url here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found the discussion in Slack
copy current revision's uri:
copy any revision's uri:
TODO