Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

Provide capability to push raw events #65

Merged

Conversation

tomkerkhove
Copy link
Contributor

Provide capability to push raw events.

Closes #57

Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
@tomkerkhove
Copy link
Contributor Author

Interpretation of EventType & DataVersion are not working as it should, will fix it and provide tests

@tomkerkhove
Copy link
Contributor Author

This is done but I've created #66 as this is not ideal.

Feel free to let me know what you think @SamVanhoutte & @stijnmoreels

@stijnmoreels
Copy link
Member

Sorry @tomkerkhove for the review delay. It's almost February! 😊

@tomkerkhove
Copy link
Contributor Author

Not sure if I'm following, what does February have to do with that? :)

@tomkerkhove
Copy link
Contributor Author

Given I need this for arcus-azure/arcus.eventgrid.proxy#29 and it works + nothing breaking I'll merge this in but feel free to do review this and I'll do a follow-up PR

@tomkerkhove tomkerkhove merged commit f713026 into arcus-azure:master Jan 24, 2019
@tomkerkhove tomkerkhove deleted the features-publish-raw-events branch January 24, 2019 21:12
tomkerkhove added a commit to arcus-azure/arcus.eventgrid.proxy that referenced this pull request Jan 28, 2019
- Provide capability to push events to Azure Event Grid via API
- Provide unit tests
- Provide integration tests
- Validation for configuration

Relates to #24 	and running integration tests in CI will be done via #28 	.

This PR depends on arcus-azure/arcus.eventgrid#65 as it's now using the preview package on MyGet and should be the official one instead.
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.

2 participants