Skip to content

Simplify AmpMustache API#894

Open
sebastianbenz wants to merge 2 commits intoampstart-2.0from
amp-mustache
Open

Simplify AmpMustache API#894
sebastianbenz wants to merge 2 commits intoampstart-2.0from
amp-mustache

Conversation

@sebastianbenz
Copy link
Collaborator

  • Remove mustache directive abstraction (Section, ...) to keep the API
    simple
  • Simplify server-side template rendering
<div data-amp-bind-hidden='context != undefined'>
  {AmpMustache.render(template, initialItems)}
</div>

* Remove mustache directive abstraction (Section, ...) to keep the API
simple
* Simplify server-side template rendering

```
<div data-amp-bind-hidden='context != undefined'>
  {AmpMustache.render(template, initialItems)}
</div>
```
* fix context indirection
* update readme
@CLAassistant
Copy link

CLAassistant commented Oct 8, 2020

CLA assistant check
All committers have signed the CLA.

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