Skip to content
This repository has been archived by the owner on Mar 20, 2021. It is now read-only.

Docs on helpers incomplete #274

Closed
mulderp opened this issue Nov 18, 2013 · 6 comments
Closed

Docs on helpers incomplete #274

mulderp opened this issue Nov 18, 2013 · 6 comments

Comments

@mulderp
Copy link

mulderp commented Nov 18, 2013

Hi!

Just a small note that the (new ?) docs on the helpers are incomplete:

http://walmartlabs.github.io/thorax-docs/helpers.html

in the URL, there should probably be {{url "article/{{url}}" expand-tokens=true}}

PS - would you accept a PR for this? where would I find the docs?

@alexlande
Copy link

Those are in fact very old docs. @eastridge: probably want to delete the gh-pages branch of the thorax-docs repo.

@eriktrom
Copy link
Contributor

eriktrom commented Dec 8, 2013

@alexlande nice catch, thanks for clarifying

@mulderp new docs for http://walmartlabs.github.io/thorax-docs/helpers.html can be found at http://thoraxjs.org/api.html#template-helpers (i'm sure you found it by now though)

@eastridge - delete https://github.com/walmartlabs/thorax-docs/tree/gh-pages real quick, it's way old... if there is stuff there you want to keep, let me know, i can pull it over (also keep in mind redirecting)

also, if someone is interested in making doc changes, should they submit a PR to thorax or thorax-docs?

@alexlande
Copy link

Submit doc PRs to thorax. thorax-docs pulls in content from the Thorax README through a grunt task (this setup is going to change to something a bit cleaner sometime in the future though).

@eriktrom
Copy link
Contributor

eriktrom commented Dec 8, 2013

awesome, thankyou @alexlande

@mulderp
Copy link
Author

mulderp commented Dec 8, 2013

Thanks for the clarifications, works great!

@mulderp mulderp closed this as completed Dec 8, 2013
@eastridge
Copy link
Contributor

Deleted the gh-pages branch thanks everyone for catching this

On Sun, Dec 8, 2013 at 2:05 AM, Patrick Mulder [email protected]:

Closed #274 #274.


Reply to this email directly or view it on GitHubhttps://github.com//issues/274
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

4 participants