geneontology / termgenie

Web-application for pattern-based ontology class generation
7 stars 6 forks source link

Write documentation for new TG login system #101

Closed mcourtot closed 7 years ago

mcourtot commented 7 years ago

as per https://github.com/geneontology/termgenie/issues/98

mcourtot commented 7 years ago

https://docs.google.com/document/d/1DWgeOy1ICvBAW3xOf18wjGiOK3dg0uOw82Q20mvEbj8/edit

@cmungall, @kltm, @nathandunn: please check/update as needed

@paolaroncaglia, @dosumis, @ukemi, @tberardini: please check this makes sense for users

cmungall commented 7 years ago

This is great

Will someone go ahead ad announce? Remember and thank Nathan and Seth - and help from Heiko

On Thu, Dec 1, 2016 at 7:06 AM, Melanie Courtot notifications@github.com wrote:

https://docs.google.com/document/d/1DWgeOy1ICvBAW3xOf18wjGiOK3dg0 uOw82Q20mvEbj8/edit

@cmungall https://github.com/cmungall, @kltm https://github.com/kltm, @nathandunn https://github.com/nathandunn: please check/update as needed

@paolaroncaglia https://github.com/paolaroncaglia, @dosumis https://github.com/dosumis, @ukemi https://github.com/ukemi, @tberardini https://github.com/tberardini: please check this makes sense for users

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/geneontology/termgenie/issues/101#issuecomment-264156598, or mute the thread https://github.com/notifications/unsubscribe-auth/AADGOe69ZfpxO5fBJg5zsjZ-A6CyhKCHks5rDrg9gaJpZM4LBTHD .

paolaroncaglia commented 7 years ago

Thanks @mcourtot , yes the doc makes sense to me. I'm logged into GitHub, so when I tried to log in to TG just now, I was presented with a simpler option, i.e. do I authorise access to my email address rather than a full log in request. But I don't think there's any need to tell users about that option too - if they already are on GitHub, they'll figure it out. Happy for you to announce as you wrote the doc :-)

nathandunn commented 7 years ago

@mcourtot Looks great. Other than changing the owner (from me), it looks good.

tberardini commented 7 years ago

Makes sense to me!

ukemi commented 7 years ago

I made one tiny edit, but it makes sense to me too.

kltm commented 7 years ago

Okay, I'm making a lot of edits, and it's going too well. I don't think this will work real well as a google doc as there is too much structured into (and it's nearly impossible to get YAML to layout right).

kltm commented 7 years ago

Great document though. Would anybody mind if I tried to make this a little more permanent and added this to the go-site repo as a markdown doc, similar to what we have for the Noctua quickstart guide?

kltm commented 7 years ago

And maybe generalize it a bit to include Noctua, as they are essentially the same in this aspect. As well, this would be a chance to push for people to get ORCIDs. Some kind of uri is required anyways. Do people think a merged doc works for this? https://github.com/geneontology/go-site/blob/master/documentation/application-logins.md (I'm not happy how it lays out in go-site like that, but I can't think of a better place to leave it for now...)

mcourtot commented 7 years ago

That's amazing, thanks for all the updates @kltm !

I used this to send an email to go-discuss announcing the changes, and am now closing this ticket.