phetsims / number-line-distance

"Number Line: Distance" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
0 stars 3 forks source link

Finalize Credits #39

Closed SaurabhTotey closed 3 years ago

SaurabhTotey commented 3 years ago

Standard code review issue created for #35. Should be addressed before publication.

pixelzoom commented 3 years ago

As of code review https://github.com/phetsims/number-line-distance/issues/35, the credits are completed empty. It would be good to take a stab at filling these in before the first dev test. At this point in the development, you should know everything except QA credits.

SaurabhTotey commented 3 years ago

Hey @amanda-phet, an RC was recently created with an attempt at filling out credits. Do these credits look good to you?

jbphet commented 3 years ago

When asked about credits for graphic arts, @amanda-phet said (over Slack):

Mariah Hermsmeyer, Megan Lai, Amanda McGarry

When asked about other designers, she said:

Kathy Perkins, Ian Whitacre

SaurabhTotey commented 3 years ago

The credits should be complete, I would just like them to be verified.

stemilymill commented 3 years ago

When asked about credits for graphic arts, @amanda-phet said (over Slack):

Mariah Hermsmeyer, Megan Lai, Amanda McGarry

When asked about other designers, she said:

Kathy Perkins, Ian Whitacre

Based on this comment, Amanda should be included in the graphic arts credits and she is currently not.

https://github.com/phetsims/qa/issues/695

SaurabhTotey commented 3 years ago

I didn't put Amanda in the graphic arts section because she is already in the lead design section. @jbphet told me not to duplicate the name in both sections, but I asked in slack to make sure, where @pixelzoom replied "The policy is indeed no duplication. There’s no need to put Amanda in graphicArts." As such, I will close this issue, but please reopen if there are any other issues with credits.

SaurabhTotey commented 3 years ago

Reopening because @pixelzoom pointed out that this issue should stay open until the sim is published as QA credits may change.

KatieWoe commented 3 years ago

I believe QA credits are fine for this