search

nih-cfde / organization

Main repository for meta information about the organization
Other
7 stars 8 forks source link

Update Kids First , UDN, IDG #63

Closed josesanchez1815 closed 4 years ago

josesanchez1815 commented 4 years ago

Updating Principal Investigators/Project Managers for KF, UDN, IDG I need some help fixing some of the boxes that do not have a line to distinguish the rows.

ACharbonneau commented 4 years ago

Why does KF have a company column now?

josesanchez1815 commented 4 years ago

Oh sorry! I didn’t clarify. Within Kids First, there are several what looked like subgroups within the Kids First. I put company as a way to distinguish them. There is CHOP, Ste. Justine, and Seven Bridges. Each has a lead person and project manager. I thought distinguishing them would have been helpful. I thought it would have been confusing to have multiple PIs and multiple project managers. But if that’s weird you don’t have to use it.

On Mon, Oct 26, 2020 at 1:59 PM Jeremy Walter notifications@github.com wrote:

@jeremywalter commented on this pull request.

I think there is some confusion here (some of these people are already in the POC). I will cross check and make a new PR. I'm not clear on why Company has been added either...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nih-cfde/organization/pull/63#pullrequestreview-517170818, or unsubscribe https://github.com/notifications/unsubscribe-auth/APAA6SV4RLNKX4EPFE5KHM3SMXPK3ANCNFSM4S5F3Y2A .

jeremywalter commented 4 years ago

Ah, I see... I think it depends on how this list is to be used. Up to now it's just been contact info and loose association for people with the different DCC projects. I think the Master List has more detail that might not be needed here, but I don't actively use this list, so I don't know. If we're adding another cell I think it needs to be everywhere for consistency. @ACharbonneau ?

josesanchez1815 commented 4 years ago

That is true. We can let let the master contact info be our primary source of accessing contacts.

ACharbonneau commented 4 years ago

I am not about to tell you that you can't sub-divide people in that way, but it's going to require extensive reformatting of the entire thing, and for you do to a lot of going to get the extra affiliations from everyone else...KF is absolutely not the only multi-institution group. I would vote for no on the extra info, but if you guys want it, and are going to update the rest of them to match, then it's fine. I wouldn't call it Company tho.

josesanchez1815 commented 4 years ago

Oh if that's the case then..... we can ignore my edits. I don't really want to update the point of contacts with extra information that can be found in the master list.

josesanchez1815 commented 4 years ago

Removed Company Column as it is unnecessary at this time.