Skip to content

Clarify options for resolving template variables #291

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

Closed
wants to merge 1 commit into from

Conversation

handrews
Copy link
Contributor

Addresses #288

If a variable can be resolved from either the instance or from
client data, the instance data may be used as a default when
asking for client data.

Also clarify how "hrefSchema" can handle a server-computed
URI path with client-supplied data string, as in traditional
HTML "get" forms. This should address @awwright's
#288 (comment)

If a variable can be resolved from either the instance or from
client data, the instance data may be used as a default when
asking for client data.
@handrews handrews added this to the draft-next (draft-6) milestone Mar 31, 2017
@jdesrosiers
Copy link
Member

I might be missing something, but I don't see how this changes anything. That dependency is still there. If anything this just seems to acknowledge that the dependency exists.

@handrews
Copy link
Contributor Author

Given the state of things in #288, I am inclined to discard this unless @awwright or someone else thinks it is useful independent of the concerns in that issue.

@handrews
Copy link
Contributor Author

handrews commented Apr 3, 2017

I've decided that this is probably just confusing, and given that we don't have consensus on this as a desired interpretation it's better to leave it open and see what sort of feedback we get. In particular, we already have several discussions going around default values, and hopefully one of those will result in more philosophically consistent solutions.

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.

2 participants