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

Add a ImportEvent and ExportEvent #50

Merged
merged 2 commits into from
Oct 5, 2015

Conversation

benjy
Copy link
Collaborator

@benjy benjy commented Oct 1, 2015

No description provided.

@jibran
Copy link
Collaborator

jibran commented Oct 1, 2015

Can we have some tests?

@benjy
Copy link
Collaborator Author

benjy commented Oct 1, 2015

What kind of test do you propose?

@larowlan
Copy link
Owner

larowlan commented Oct 1, 2015

Unit test is fine
On 1 Oct 2015 6:20 pm, "Ben Dougherty" notifications@github.com wrote:

What kind of test do you propose?


Reply to this email directly or view it on GitHub
#50 (comment)
.

use Drupal\rest\LinkManager\LinkManagerInterface;
use Drupal\rest\Plugin\Type\ResourcePluginManager;
use Symfony\Component\EventDispatcher\Event;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not used

@larowlan
Copy link
Owner

larowlan commented Oct 1, 2015

Happy for merge - tests can come in follow up - thanks @benjy

benjy added a commit that referenced this pull request Oct 5, 2015
@benjy benjy merged commit 80fb60d into larowlan:8.x Oct 5, 2015
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.

None yet

3 participants