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

Register section for the app in settings #62

Merged
merged 1 commit into from
May 11, 2017

Conversation

sharidas
Copy link
Contributor

Register section for the app in settings

Signed-off-by: Sujith H [email protected]

@CLAassistant
Copy link

CLAassistant commented May 10, 2017

CLA assistant check
All committers have signed the CLA.

@sharidas sharidas requested a review from PVince81 May 10, 2017 16:00
@PVince81
Copy link
Contributor

Thanks 👍

@sharidas sharidas force-pushed the customgroups-consistancy-task16 branch 3 times, most recently from 46563b3 to 52c00ff Compare May 11, 2017 05:44
@PVince81
Copy link
Contributor

@SergioBertolinSG why is it failing for mailhog here even though we don't care about email here ?

 [Behat\Testwork\Argument\Exception\UnknownParameterValueException]                                                

  Can not find a matching value for an argument `$mailhog_url` of the method `CustomGroupsContext::__construct()`.

@SergioBertolinSG
Copy link
Contributor

Because tests here are directly affected by changes in core.

That variable is missing in the config.yml I can try to fix it.

@SergioBertolinSG
Copy link
Contributor

Modified here #63 .

@PVince81
Copy link
Contributor

@sharidas please rebase for CI

@PVince81
Copy link
Contributor

then merge if it passes

Register section for the app in settings

Signed-off-by: Sujith H <[email protected]>
@sharidas sharidas force-pushed the customgroups-consistancy-task16 branch from 52c00ff to 27c43fd Compare May 11, 2017 10:08
@sharidas
Copy link
Contributor Author

@PVince81 sure

@sharidas sharidas merged commit e2b6c1f into master May 11, 2017
@sharidas sharidas deleted the customgroups-consistancy-task16 branch May 11, 2017 10:51
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.

4 participants