Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added register transports forms Issue#44 #81

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

Shashika
Copy link
Contributor

No description provided.

@gayanhewa
Copy link
Contributor

@Shashika does this PR relate to the invisible captcha PR looks like the work overlaps?

@Shashika
Copy link
Contributor Author

Shashika commented May 29, 2017

@gayanhewa Yes, it is my fault, previously I added PR to master mistakenly. d2c37e9, 5f3f060, 28358d0 commits are already in master but not in dev.

@chamilad
Copy link
Contributor

Also, it would be a good practice to mention something like Resolves #{issueNumber} in the PR description to automatically close the issue associated with this PR once this is merged.

@Shashika Shashika mentioned this pull request Jun 2, 2017
@danishka
Copy link
Collaborator

danishka commented Jun 9, 2017

@thinkholic @naveendw can we merge this?

@naveenmadhawa
Copy link
Collaborator

@Shashika can you fix these conflict

# Conflicts:
#	.env.example
#	flood_2017-05-27.sql
#	resources/views/layouts/master.blade.php
@Shashika
Copy link
Contributor Author

@naveenmadhawa Fixed conflicts. Have a look.

@danishka
Copy link
Collaborator

@thinkholic ^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants