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

Add PingMessage definition (PNX-137) #70

Closed
wants to merge 1 commit into from

Conversation

russellpkb
Copy link
Contributor

Simple avro definition for ping messages used in pubsub for communicating between phoenix and phr

@russellpkb russellpkb added the dont-merge-yet Don't merge this yet label Sep 18, 2021
Copy link

@pkbdevdeploy pkbdevdeploy left a comment

Choose a reason for hiding this comment

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

Version 40-head-b62011d-163585-SNAPSHOT was generated from commit b62011db02e0aa0fc2af21ee36277b4c8ac218e6

Copy link

@pkbdevdeploy pkbdevdeploy left a comment

Choose a reason for hiding this comment

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

Version 40-head-f1f2ae9-163734-SNAPSHOT was generated from commit f1f2ae9ea6185ae8a46eddb459a46cc26192d8a9

@russellpkb russellpkb force-pushed the feature/PNX-137-pubsub-message branch from f1f2ae9 to 8a54450 Compare October 5, 2021 11:57
Copy link

@pkbdevdeploy pkbdevdeploy left a comment

Choose a reason for hiding this comment

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

Version 42-head-8a54450-167166-SNAPSHOT was generated from commit 8a544502909b518072b4228f319425dd4a1c6252

@MFAshby
Copy link
Contributor

MFAshby commented Oct 28, 2021

this is PoC work and doesnt' need to be merged 👍 new PR will be raised with the actual messages we require.

@MFAshby MFAshby closed this Oct 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dont-merge-yet Don't merge this yet
Projects
None yet
3 participants