Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

fixed #52: AuthorizationHandler mapped to wrong factory #53

Merged
merged 1 commit into from
Oct 31, 2018
Merged

fixed #52: AuthorizationHandler mapped to wrong factory #53

merged 1 commit into from
Oct 31, 2018

Conversation

marc-mabe
Copy link
Member

Provide a narrative description of what you are trying to accomplish:

  • Are you fixing a bug?

  • Are you creating a new feature?

    • Why is the new feature needed? What purpose does it serve?
    • How will users use the new feature?
    • Base your feature on the develop branch, and submit against that branch.
    • Add only one feature per pull request; split multiple features over multiple pull requests
    • Add tests for the new feature.
    • Add documentation for the new feature.
    • Add a CHANGELOG.md entry for the new feature.
  • Is this related to quality assurance?

  • Is this related to documentation?

@ezimuel
Copy link
Contributor

ezimuel commented Oct 31, 2018

@marc-mabe thanks for this PR, this remind me that we MUST test the ConfigProvider!

@ezimuel ezimuel merged commit e5ccc66 into zendframework:master Oct 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants