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 Markdown previewer in Guide #9986

Merged
merged 6 commits into from
Oct 11, 2023
Merged

Add Markdown previewer in Guide #9986

merged 6 commits into from
Oct 11, 2023

Conversation

mr-loop-1
Copy link
Contributor

@mr-loop-1 mr-loop-1 commented Oct 9, 2023

What this PR does / why we need it:

The guide needs to be updated for new addition of markdown previewer in list of file previewers and external tools.

Which issue(s) this PR closes:

Special notes for your reviewer: Added markdown after pdf based on probable frequency among types

Suggestions on how to test this: NA

Does this PR introduce a user interface change? If mockups are available, please link/include them here: No

Is there a release notes update needed for this change?: No

Additional documentation: No

@mr-loop-1 mr-loop-1 changed the title 9412 update md previewer in guide Add Markdown previewer in Guide Oct 9, 2023
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

@mr-loop-1 thanks for the pull request! 🎉 ❤️

I added a release note. I think I forgot to mention that, sorry. That process is at https://guides.dataverse.org/en/6.0/developers/making-releases.html#write-release-notes and there's an open issue for that too, if you're interested in making another PR:

I'm leaving a couple ```suggestion comments. You can just click them and it will make the change automatically. Thanks!

@pdurbin pdurbin self-assigned this Oct 9, 2023
mr-loop-1 and others added 2 commits October 9, 2023 19:03
Co-authored-by: Philip Durbin <[email protected]>
@mr-loop-1
Copy link
Contributor Author

mr-loop-1 commented Oct 9, 2023

Thanks a lot @pdurbin ,
Quite Comically, I was of another impression about the work in this issue.

@pdurbin pdurbin added the hacktoberfest-accepted Accepted for HF, see https://groups.google.com/g/dataverse-community/c/n_Nn_T2yA-w/m/BcoXO4tEAQAJ label Oct 9, 2023
@pdurbin
Copy link
Member

pdurbin commented Oct 9, 2023

@mr-loop-1 looks great! I'll talk about it during standup tomorrow (11am Eastern) and approve it then. Thanks!

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks! Approved! 🎉

@pdurbin pdurbin removed their assignment Oct 10, 2023
@kcondon kcondon self-assigned this Oct 11, 2023
@kcondon kcondon merged commit c1c984d into IQSS:develop Oct 11, 2023
@pdurbin pdurbin added this to the 6.1 milestone Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted for HF, see https://groups.google.com/g/dataverse-community/c/n_Nn_T2yA-w/m/BcoXO4tEAQAJ
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Markdown previewer to list in Admin Guide
3 participants