Skip to content

2.x#32

Merged
bancer merged 18 commits intomasterfrom
2.x
Mar 8, 2017
Merged

2.x#32
bancer merged 18 commits intomasterfrom
2.x

Conversation

@bancer
Copy link
Owner

@bancer bancer commented Mar 8, 2017

No description provided.

bancer and others added 18 commits February 27, 2017 11:32
…iable (#22)

* Wizard root session key is refactored to a configurable component
variable.

* fixed unit test

* Moved sessionRootKey related code from initialize to startup to ease the
configuration in a controller.

* Setting session keys refactored to a private method.

* Fixed variable name.
redirects-optimization-conflicts

Conflicts:
	Controller/Component/WizardComponent.php
Some refactoring in order to reduce the number of redirects
…nto redirects-optimization-conflicts

Conflicts:
	Controller/Component/WizardComponent.php
redirects-optimization

Conflicts:
	Controller/Component/WizardComponent.php
@codecov-io
Copy link

codecov-io commented Mar 8, 2017

Codecov Report

Merging #32 into master will decrease coverage by -1.2%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master     #32     +/-   ##
========================================
- Coverage    77.5%   76.3%   -1.2%     
========================================
  Files           2       2             
  Lines         280     287      +7     
========================================
+ Hits          217     219      +2     
- Misses         63      68      +5
Impacted Files Coverage Δ
Controller/Component/WizardComponent.php 70.75% <100%> (-1.45%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96b2409...358b876. Read the comment docs.

@bancer bancer merged commit f5239cd into master Mar 8, 2017
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.

2 participants