Skip to content
This repository has been archived by the owner on Dec 1, 2020. It is now read-only.

Api documentation #147

Merged
merged 3 commits into from
Nov 14, 2015
Merged

Api documentation #147

merged 3 commits into from
Nov 14, 2015

Conversation

jwasilgeo
Copy link
Contributor

@tomwayson please review. This is for #99 (comment). The checklist as of creating this PR should reflect what has been added to the documentation.

I also added some cross-referenced links between docs and some site example pages.

I couldn't get the ngdocs to generate content for the esri.map and esri.core docs pages, and also not fully sure if that is even a feature.

Feel free to edit heavily; some params and descriptions I really wasn't 💯% sure on and would love your feedback.

@tomwayson
Copy link
Member

@jwasil I'm realizing that I'd like to go ahead and merge this b/c I'm going to need to update esriLoader to add .requireSync() and I'd rather avoid the merge conflicts. Let me know if you're good for me to merge and then you can open a new branch for remaining docs.

@jwasilgeo
Copy link
Contributor Author

Please go ahead. I'm away from a computer right now, otherwise I'd jump in and help out.

tomwayson added a commit that referenced this pull request Nov 14, 2015
@tomwayson tomwayson merged commit 5f71ba3 into master Nov 14, 2015
@tomwayson tomwayson deleted the api-documentation branch November 15, 2015 20:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants