search

Picolab / devtools

Repository for KRL Developer Tools
MIT License
6 stars 4 forks source link

Change names in subscription list #90

Closed windley closed 8 years ago

windley commented 8 years ago

We're using Event Channel and Back Channel instead of inbound and outbound

screenshot 2016-04-22 13 00 45

AngelSassin commented 8 years ago

Event Channel and Target Channel are used in the Inbound and Outbound subscriptions lists. Should they also be renamed to Inbound and Outbound?

Inbound Subscriptions List: uploads/cfa9d896-60e2-4e7c-94c4-8ed451f98256/Screen Shot 2016-05-06 at 4.36.38 PM.png

Outbound Subscriptions List: uploads/b7327bcb-dec2-4411-919a-c77f73aa694b/Screen Shot 2016-05-06 at 4.36.24 PM.png

windley commented 8 years ago

Leave them for now and let’s look at them.

On May 6, 2016, at 4:47 PM, Nicholas Angell notifications@github.com wrote:

Event Channel and Target Channel are used in the Inbound and Outbound subscriptions lists. Should they also be renamed to Inbound and Outbound?

Inbound Subscriptions List: https://camo.githubusercontent.com/cf823f46ec09daf9690bcf666edafc5de377de1e/68747470733a2f2f73332d75732d776573742d322e616d617a6f6e6177732e636f6d2f70726f642e6875626f6172642e636f6d2f75706c6f61647325324663666139643839362d363065322d346537632d393463342d38656434353166393832353625324653637265656e2b53686f742b323031362d30352d30362b61742b342e33362e33382b504d2e706e67 Outbound Subscriptions List: https://camo.githubusercontent.com/8bad2e807dec6aab362d73220ccf0b9bdaf95d06/68747470733a2f2f73332d75732d776573742d322e616d617a6f6e6177732e636f6d2f70726f642e6875626f6172642e636f6d2f75706c6f61647325324662373332376263622d646563322d343431312d393139612d63373766373361613639346225324653637265656e2b53686f742b323031362d30352d30362b61742b342e33362e32342b504d2e706e67 — You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/kre/devtools/issues/90#issuecomment-217581696

windley commented 8 years ago

Change Event Channel and Back Channel in subscription list to inbound and outbound. For Inbound list, leave "Event Channe". For Outbound list, change "Target" to "Event"

eisbaerBorealis commented 8 years ago

Issue 14 on wrangler says to change Event Channel to Outbound and Back Channel to Inbound, which is the opposite of what is said here. Is that correct?

windley commented 8 years ago

Well, that's why we're changing them. :) Ed's terminology is confusing. The inbound channel is the one that the pics owns. I believe that is the back channel in the old terminology.  The outbound channel is the one that we use to send events to the other pico. I believe that is the target or event channel in the old lexicon.  Feel free to argue with me if you think it is different. 

On Wed, May 18, 2016 at 11:39 AM -0700, "Jesse Howell" notifications@github.com wrote:

Issue 14 on wrangler says to change Event Channel to Outbound and Back Channel to Inbound, which is the opposite of what is said here. Is that correct?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub

eisbaerBorealis commented 8 years ago

On the Subscriptions list, I changed "Event Channel" to "Outbound" and "Back Channel" to "Inbound". On the list of Inbound Subscriptions, I changed nothing. On the list of Outbound Subscriptions, I changed "Target Channel" to "Event Channel".