/interface-juju-info

Update the README for better documented example usage of the interface #3

Merged
merged 1 commit into from Aug 8, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@chuckbutler @kwmonroe @paulgear
@chuckbutler
Collaborator

chuckbutler commented Aug 8, 2016

No description provided.

@chuckbutler chuckbutler referenced this pull request in juju/docs Aug 8, 2016

Open

interface-layers needs expanded documentation on {{requires:foo}}-relation-{{joined,changed}} #1288

@chuckbutler
Collaborator

chuckbutler commented Aug 8, 2016

Additional information:

Furthermore, the juju-info interface layer just lists {{name}} in the readme, assuming the reader knows exactly which magic string should be placed there. This is one of the most opaque and confusing barriers to using reactive relations, and needs tooling and documentation to guide new users.
@kwmonroe
Member

kwmonroe commented Aug 8, 2016

Nice @chuckbutler! LGTM

@kwmonroe kwmonroe merged commit 19bc16f into juju-solutions:master Aug 8, 2016

@spads-spads spads-spads referenced this pull request in juju/docs Aug 9, 2016

Open

Charm Developer: Need best practice page on reactive patterns and anti-patterns #1263

@paulgear

paulgear commented Aug 9, 2016

Presumably "conneted" should read "connected"?
@chuckbutler
Collaborator

chuckbutler commented Aug 9, 2016

addressed in #4, thanks @paulgear
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment