Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add link to the full note url #12

Merged
merged 2 commits into from
Dec 21, 2020
Merged

add link to the full note url #12

merged 2 commits into from
Dec 21, 2020

Conversation

lanjelot
Copy link
Collaborator

the note URL is now clickable for convenience

@@ -77,7 +77,7 @@
</li>

<li class="list-group-item list-group-item-action">
<strong>Note Id:</strong> <code>{{challenge.note_id}}</code>
<strong>Note:</strong> <code><a href="{{challenge.note_url}}">{{challenge.note_id}}</a></code>
Copy link
Owner

Choose a reason for hiding this comment

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

Make the link a target="_blank"

@lanjelot lanjelot requested a review from hugsy December 21, 2020 06:58
@hugsy hugsy merged commit e35f660 into hugsy:master Dec 21, 2020
@lanjelot lanjelot deleted the full-note-url branch December 22, 2020 07:02
@hugsy hugsy added this to the 0.1 milestone Jun 21, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants