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
mention jsonapi-rb for people specifically looking for jsonapi serialization #2195
Conversation
README.md
Outdated
|
||
In the mean time, if you're looking for a json:api renderer, [jsonapi-rb](http://jsonapi-rb.org/) will meet your needs -- | ||
with a [performance boost](https://gist.github.com/NullVoxPopuli/748e89ddc1732b42fdf42435d773734a), too. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe instead of adding this text here, add here a link to [Development Status](#status-of-ams)
and then put this down there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And maybe reference https://github.com/rails-api/active_model_serializers/tree/benchmarks per discussion in #2169 ??
README.md
Outdated
@@ -41,6 +43,13 @@ Please see below for the documentation relevant to you. | |||
- ❗️ All existing PRs against master will need to be closed and re-opened against 0-10-stable, if so desired | |||
- ❗️ Master, for the moment, won't have any released version of AMS on it. | |||
|
|||
In the mean time, if you're looking for a json:api renderer, [jsonapi-rb](http://jsonapi-rb.org/) will meet your needs -- | |||
with a [performance boost](https://gist.github.com/NullVoxPopuli/748e89ddc1732b42fdf42435d773734a), too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe something like?
### Alternatives
- [jsonapi-rb](http://jsonapi-rb.org/) is a [highly performant](https://gist.github.com/NullVoxPopuli/748e89ddc1732b42fdf42435d773734a) and modular JSON:API-only implementation. There's a vibrant community around it that has produced projects such as [JSON:API Suite](https://jsonapi-suite.github.io/jsonapi_suite/).
For benchmarks against alternatives, see https://github.com/rails-api/active_model_serializers/tree/benchmarks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
works for me!
README.md
Outdated
- [jsonapi-rb](http://jsonapi-rb.org/) is a [highly performant](https://gist.github.com/NullVoxPopuli/748e89ddc1732b42fdf42435d773734a) and modular JSON:API-only implementation. There's a vibrant community around it that has produced projects such as [JSON:API Suite](https://jsonapi-suite.github.io/jsonapi_suite/). | ||
|
||
For benchmarks against alternatives, see https://github.com/rails-api/active_model_serializers/tree/benchmarks | ||
|
||
*Changes to 0.10.x maintenance*: | ||
|
||
- The 0.10.x version has become a huge maintenance version. We had hoped to get it in shape for a 1.0 release, but it is clear that isn't going to happen. Almost none of the maintainers from 0.8, 0.9, or earlier 0.10 are still working on AMS. We'll continue to maintain 0.10.x on the 0-10-stable branch, but maintainers won't otherwise be actively developing on it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/version/burden
lolz
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:-D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I should probably actually have a different subheader for this stuff
README.md
Outdated
@@ -2,6 +2,8 @@ | |||
|
|||
## About | |||
|
|||
ActiveModelSerializers is undergoing some renovations. see [Development Status](#status-of-ams) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing .
capitalize See
## Installation | ||
|
||
## Getting Started | ||
ActiveModelSerializers is undergoing some renovations. See [Development Status](#status-of-ams). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure
README.md
Outdated
|
||
*Changes to 0.10.x maintenance*: | ||
|
||
### *Changes to 0.10.x maintenance*: | ||
|
||
- The 0.10.x version has become a huge maintenance version. We had hoped to get it in shape for a 1.0 release, but it is clear that isn't going to happen. Almost none of the maintainers from 0.8, 0.9, or earlier 0.10 are still working on AMS. We'll continue to maintain 0.10.x on the 0-10-stable branch, but maintainers won't otherwise be actively developing on it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would you mind fixing the typo whilst here? s/version/burden? and add a period to end of bullet line?
README.md
Outdated
- [0.10 (0-10-stable) Documentation](https://github.com/rails-api/active_model_serializers/tree/0-10-stable) | ||
- [0.10.6 (latest release) Documentation](https://github.com/rails-api/active_model_serializers/tree/v0.10.6) | ||
- [![API Docs](http://img.shields.io/badge/yard-docs-blue.svg)](http://www.rubydoc.info/gems/active_model_serializers/0.10.6) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since you're making changes, I guess remove the 0.10.6 stuff and make more in line with 0-9 and 0-8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can do
README.md
Outdated
- [![API Docs](http://img.shields.io/badge/yard-docs-blue.svg)](http://www.rubydoc.info/github/rails-api/active_model_serializers/0-8-stable) | ||
<table> | ||
<tr> | ||
<td> [0.10 (0-10-stable) Documentation](https://github.com/rails-api/active_model_serializers/tree/0-10-stable) </td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks better in atom. :-\
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- [![API Docs](http://img.shields.io/badge/yard-docs-blue.svg)](http://www.rubydoc.info/github/rails-api/active_model_serializers/0-8-stable) | ||
<table> | ||
<tr> | ||
<td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
html. blegh. but the result is nice.
Just a simple note.