Note: Make this feature configurable through a checkbox in the config page

Reference: http://dashinotify.com

The purpose of Dashi in GitHub is to be an open-source project that grows with developers contributions.
The readme should be a dedicated guide to help developers to start contributing with Dashi.
As Dashi now have a dedicated web page that users can use to set up and start using the service, is a good chance to improve the readme structure with something more comfortable to developers.
I'm going to work on this.

When the background of the login page is loading, we get duplicated login form.

Sometimes we just let a comment in the PR and we don't mention someone for some reason. The thing is that we should send a notification to all the participants (not reviewers) excluding the author of the comment and people who were mentioned in the comment (in order to avoid duplicated notifications).

Currently, you can send anything in the token field, we should validate it.

A token look like this:

• https://hooks.slack.com/services/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

As you can see the pattern is:

• https://hooks.slack.com/services/ + 9 alphanumeric + / + 9 alphanumeric + / + 24 alphanumeric

We should validate this pattern, currently we only make sure that the user sent something in this field
app/Http/Controllers/HomeController.php#L35

• Use a regex expression to validate this

Notifications are getting duplicated when more than one reviewer is selected

Change each dashi icons that represent whats going on in a pull request.

• Create a Purple Dashi icon for Merge Request. #34
• Create a Blue Dashi icon for information purpose. #34
• Change dashi icons use.

As follows:

Green Dashi Icon:

• PR Review

Purple Dashi Icon:

• PR Merged

Yellow Dashi Icon:

• PR Changes

Red Dashi icon:

• Denied PR

Blue Dashi icon:

• Mentioned in a PR

Something like this

Branch
development -> master

• Review Requested
• Merged
• Rejected
• Updated
• Changes Requested

Make the setup project steps an interactive video to post in the home web page.

Look for another picture provider API.
"Look at this API, it looks like... shit."

We need to add unit testing to the project because it is getting hard to test all the features one by one for each release.

Add support to send a notification when a pull request is merged.

• Add username and avatar
• Long fields
• Add meta info (dashi logo and timestamp)
• Custom label color

Example

• Github #62
• Bitbucket #66

Something like this success icon but in yellow

This should be done with:

• Github
• Bitbucket

We need this to be able to send emails such as the Forgot Your Password

Notes:

• You should enable the push webhook to archive this
• Only pull requests with an open state should trigger the notification

The PR owner is @DannyFeliz08 and who requested the changes is @gabbanaesteban, so this notification should be displaying DannyFeliz instead of gabbanaesteban

Consider check the fields:

"action": "closed",
"merged": true,
"merged_by": null

in order to check if the PR was closed explicitly or merged