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

Add ResourceEvent To EVENT_FETCH_PRE #316

Merged
merged 2 commits into from
Dec 3, 2018
Merged

Add ResourceEvent To EVENT_FETCH_PRE #316

merged 2 commits into from
Dec 3, 2018

Conversation

datasage
Copy link
Contributor

@datasage datasage commented Dec 3, 2018

This pull request adds \ZF\Rest\ResourceEvent to EVENT_FETCH_PRE event.

@datasage datasage mentioned this pull request Dec 3, 2018
Closed
@coveralls
Copy link

coveralls commented Dec 3, 2018 •
edited
Loading

Coverage Status

Coverage increased (+0.03%) to 53.951% when pulling 20b5070 on datasage:datasage-fetch-pre into 6d45064 on zfcampus:master.

@michalbundyra
Copy link
Member

@datasage We need test case for that change, thanks!

@TomHAnderson
Copy link
Contributor

@datasage HMU if you need help with that.

@weierophinney
Copy link
Member

Finding where to test this was tricky, so I've added a unit test for this and pushed to the branch created by @datasage

@weierophinney weierophinney merged commit 20b5070 into zfcampus:master Dec 3, 2018
weierophinney added a commit that referenced this pull request Dec 3, 2018
@weierophinney
Adds CHANGELOG entry for #316
420ce0e
weierophinney added a commit that referenced this pull request Dec 3, 2018
@weierophinney
Merge branch 'hotfix/316-resource-event-pre-fetch' into develop
1413db8 
Forward port #316
Fixes #315

Conflicts:
	CHANGELOG.md
@michalbundyra michalbundyra added this to the 2.2.1 milestone Dec 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.2.1
Development

Successfully merging this pull request may close these issues.
5 participants
@datasage @coveralls @michalbundyra @TomHAnderson @weierophinney