Add #171

[jtprichett]

Add #171: Support for accepted and rejected requests. Currently set to simple counters, serialized into an array that is passed to the Request Manager GUI. Then displayed as text objects.

[hrj]

Thanks @JoshuaTPritchett, this is a great start. Few comments:

• The accepted and rejected counts need to move into the RequestManager button.
• The counts in the button would need to be updated whenever a request is checked with the RequestManager.
• You need to sign the CLA (it helps protect your contribution as well as the project).
• Are you able to see my messages on IRC? If you don't have an IRC bouncer you can join the room via matrix.org. Else, join our gitter channel: https://gitter.im/UprootLabs/gngr

[jtprichett]

@hrj I am not sure what you mean, currently, if a user types www.google.com then it would display all of the accepted requests and all of the rejected requests. This is being displayed after you click on the request manager button. Every time a user makes a new domain request, the counters are reset and updated for the new page.

[hrj]

This is being displayed after you click on the request manager button.

The goal is to display those numbers before clicking. So I imagine them to be shown on the button itself, not in the dialog.

[hrj]

@JoshuaTPritchett Thanks for your work. I am sorry I didn't respond earlier. For one, github doesn't notify me when you update the PR, and secondly, I had just joined a new project in July, and forgot to follow up here.

The changes look good and ready to merge.

Would it be possible for you to sign the CLA? If not, I am willing to consider other alternatives such as Developer Certificate of Origin.

[hrj]

To make contributions easier, we are changing our policy to support Developer Certificate of Origin (DCO) and not requiring a CLA anymore. DCO is becoming the new standard used by many open-source projects including the Linux Kernel and GNOME.

See our Contributing Guide for more info about DCO.

[hrj]

Thanks @JoshuaTPritchett

But I can't merge without DCO 😢