Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Add nominee: Jellow AAC Communicator #961

Merged
merged 9 commits into from
Mar 22, 2022

Conversation

dpgabot
Copy link
Collaborator

@dpgabot dpgabot commented Feb 8, 2022

Automatic addition of a new nominee submitted through the online form available at https://digitalpublicgoods.net/submission

@nathanbaleeta nathanbaleeta changed the title Add nominee: jellow-aac-communicator Add nominee: Jellow AAC Communicator Feb 9, 2022
Copy link

@rahuljidgedev rahuljidgedev left a comment

Choose a reason for hiding this comment

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

..

@nathanbaleeta
Copy link
Contributor

nathanbaleeta commented Feb 24, 2022

FYI - Jellow AAC Communicator previously existed in the registry as Ninaad Digital Technology, but after getting a new reliable submission from the project owner, we have merged the old into this PR and deleted the old file.

Cc: @jwflory @conradsp

Copy link
Collaborator

@conradsp conradsp left a comment

Choose a reason for hiding this comment

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

Approve the change in name for this nominee

Copy link

@jwflory jwflory left a comment

Choose a reason for hiding this comment

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

For a nominee, the changes look okay. Seems mostly like a name change and detail update. For a full review, I have some concerns about platform independence, but we can resurface in the full review.

@nathanbaleeta nathanbaleeta merged commit dedb4d5 into main Mar 22, 2022
@nathanbaleeta nathanbaleeta deleted the jellowaaccommunicator-2f778d1cr72 branch March 22, 2022 10:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants