Skip to content
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

Starting Docs structure #994

Merged
merged 4 commits into from
Jul 17, 2015
Merged

Starting Docs structure #994

merged 4 commits into from
Jul 17, 2015

Conversation

joaomdmoura
Copy link
Member

We have being discussing about wiki and docs for a while.
It's time for us to help new and old AMS users to have a proper location to find general documentation and even 'how to' pages.

We decided to move forward with a docs/*. This PR is the initial implementation of it.

@paulodiniz
Copy link

👍

has_many :comments
has_one :author

url :post
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does url do here? :)

@bf4
Copy link
Member

bf4 commented Jul 9, 2015

Looks great! I'd be okay with merging this

@bf4
Copy link
Member

bf4 commented Jul 9, 2015

I might also want to add a section (not necessariliy in this PR) differences from 0.8 and 0.9, performance, design, extending, etc.

@joaomdmoura
Copy link
Member Author

Awesome @bf4.
Yeap, I also think we could add some sections related to older version, but the most important one would be Updating from x to 0.10.0
I'll leave this open for some time to see if anyone have more considerations or 👍

or

```ruby
ActiveModel::Serializer.config.adapter = ActiveModel::Serializer::Adapter::Json
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Above, you call it json_api but here it's json.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, I think we should stick with one convention on how to define adapters. Either class or symbol.

Or maybe something like

ActiveModel::Serializer.config.adapter = :json_api

You can also specify a class as adapter, as long as it complies with the AMS adapters interface.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which makes me think that it'd make sense to create a doc on "How to create your own adapter".

@kurko
Copy link
Member

kurko commented Jul 12, 2015

Looks good. Minor comments.

@@ -0,0 +1,26 @@
# Docs - ActiveModel::Serializer 0.10.x

This is the documentation of AMS, it's focused on **0.10.x version.**
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on the **0.10.x version**.

@joaomdmoura
Copy link
Member Author

Thank you all for reviewing it! I've updated it if all suggestions.
This is just docs but it's awesome that you could support it.
Tks also to ppl on IRC channel that +1 it.
Now it's up to all of us to add more tutorials and docs, but this is a great starting point.

joaomdmoura added a commit that referenced this pull request Jul 17, 2015
@joaomdmoura joaomdmoura merged commit 357d0d4 into rails-api:master Jul 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants