Add page about DMS Helper App

fhdsl / NIH_Data_Sharing

Learn about the new NIH data sharing policy, places where you might want to share your particular kind of data, and how to deal with possible challenges associated with the policy.

https://hutchdatascience.org/NIH_Data_Sharing/

Creative Commons Attribution 4.0 International

0 stars 0 forks source link

Add page about DMS Helper App #97

Closed avahoffman closed 1 year ago

avahoffman commented 1 year ago

Adding a page so that folks can easily find the app at https://dmshelper.fredhutch.org/
Adding a callout box to the app in the writing section
Also noticed the repo name was incorrect in _bookdown.yml

github-actions[bot] commented 1 year ago

No spelling errors! :tada: Comment updated at 2023-03-14 with changes from a3eb801349abea40e82dd8df7407f68b0d18f218

github-actions[bot] commented 1 year ago

No broken urls! :tada: Comment updated at 2023-03-14 with changes from a3eb801349abea40e82dd8df7407f68b0d18f218

github-actions[bot] commented 1 year ago

Re-rendered previews from the latest commit:

See preview of Bookdown here
See preview of Coursera/Leanpub version here
Download the preview of .docx file

Updated at 2023-03-14 with changes from a3eb801349abea40e82dd8df7407f68b0d18f218

carriewright11 commented 1 year ago

Looks great! I kinda wonder if this should go at the beginning of the writing the plan section (just seems like the most relevant spot)? I could be persuaded to keep it where it is though!

I also noticed that we use a different acronym for the plan between the app and the course - should we choose one to stay consistent? DMSP vs DSMP... I think we went with DMS initially because the NIH is calling it the Data Management and Sharing Policy. But I think DSMP is used sometimes by some tools or something

Thanks for fixing the repo link!

carriewright11 commented 1 year ago

I also noticed we have some warning messages in the about this course section after the first paragraph

avahoffman commented 1 year ago

I was hoping to make sure folks saw it early, and were aware of its existence. I don't want it to get buried.

maybe we could have a shout out in the beginning and a reminder in the writing section?

I think this might be a tad confusing, but I thought DSM Plan Helper would be too long of a url / title.

would it not just be rearranging the letters from DSMP to DMSP? maybe I am missing how that would change the url/title?

Thanks for fixing the repo link!

NP!

Addressed in https://github.com/fhdsl/NIH_Data_Sharing/pull/96

oh ok, cool

carriewright11 commented 1 year ago

Also looks like the link is to the course where the app page is- I imagine we want the link to the app?

carriewright11 commented 1 year ago

oops sorry didn't mean to edit your comment! Idk whyI did that!! little scattered

These were my responses to you:

I was hoping to make sure folks saw it early, and were aware of its existence. I don't want it to get buried.

maybe we could have a shout out in the beginning and a reminder in the writing section? (Thinking it might be good to say something like for those who are already familiar, you could skip to this app, otherwise keep reading kinda thing and then at the later point say now that you know about the policy, consider using the app?)

I think this might be a tad confusing, but I thought DSM Plan Helper would be too long of a url / title.

would it not just be rearranging the letters from DSMP to DMSP? maybe I am missing how that would change the url/title?

Addressed in https://github.com/fhdsl/NIH_Data_Sharing/pull/96

oh ok, cool

avahoffman commented 1 year ago

@carriewright11 no worries. I have to change the library name and shiny deployment so that might take a bit.

And yes, I think a shoutout at the writing stage is a good idea. will add!

avahoffman commented 1 year ago

@carriewright11 that didn't take as long as I thought 😜 How are things looking now?

carriewright11 commented 1 year ago

@carriewright11 that didn't take as long as I thought 😜 How are things looking now?

Looking very good! 😊 The name etc. on the app looks awesome! I wonder if we could make the app even more prominent in the writing section?- like maybe include a section heading and image about again? I say this because even though I knew where it was from the diff I went past it a couple of times in the preview. (I personally think the redundancy would be fine... could call it something else)

I agree that we don't want people to miss it!

Maybe also adding a statement when you introduce the app, saying for those familiar with the policy, you could skip directly to creating your plan with our app could help? Or are you afraid that will discourage people from reading the rest?

avahoffman commented 1 year ago

[x] @avahoffman Restore branch to main on cow::borrow_chapter once approved

avahoffman commented 1 year ago

@carriewright11 I recycled the content like so. Thoughts?

carriewright11 commented 1 year ago

Looks great!