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

Add doi share link and update styles #1539

Merged
merged 3 commits into from
Mar 21, 2023
Merged

Add doi share link and update styles #1539

merged 3 commits into from
Mar 21, 2023

Conversation

marySalvi
Copy link
Contributor

closes #712

Adds DOI link to share modal:
Screenshot from 2023-03-16 12-23-07

@@ -104,11 +118,12 @@ export default defineComponent({
}
return '';
});
const doiLink = computed(() => (meta.value?.doi ? `https://doi.org/:${meta.value?.doi}` : ''));
Copy link
Member

Choose a reason for hiding this comment

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

extra : snuck in. otherwise the output looks great.

Suggested change
const doiLink = computed(() => (meta.value?.doi ? `https://doi.org/:${meta.value?.doi}` : ''));
const doiLink = computed(() => (meta.value?.doi ? `https://doi.org/${meta.value?.doi}` : ''));

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed in fecc64f

Copy link
Member

@mvandenburgh mvandenburgh left a comment

Choose a reason for hiding this comment

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

One small question about an html attribute, but otherwise this looks good!

web/src/views/DandisetLandingView/ShareDialog.vue Outdated Show resolved Hide resolved
@marySalvi marySalvi added release Create a release when this pr is merged patch Increment the patch version when merged labels Mar 21, 2023
Co-authored-by: Mike VanDenburgh <37340715+mvandenburgh@users.noreply.github.com>
@marySalvi marySalvi merged commit b19d151 into master Mar 21, 2023
@marySalvi marySalvi deleted the issue-712 branch March 21, 2023 20:43
@dandibot
Copy link
Member

🚀 PR was released in v0.3.28 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update share link for published dandisets
4 participants