-
Notifications
You must be signed in to change notification settings - Fork 3
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
Organizing and Naming Serializers/Deserializers #9
Comments
I've implemented all of these in the Python library. Here's what I did.
I wanted to get something working, so I just went ahead with what I had. If we decided to name these differently, I'll gladly change any these! |
👍 I agree on every point. The only one I am not sure is
I would expect the Hypermedia Resource work out-of-the-box that is – using all means possible parse response into Hypermedia Resource objects – if that means the |
I agree with you on that, and that's the direction I took. I do have a "browser" adapter for building HTML, but I'm not including that by default at this point. Right now, if the user imports the hypermedia library, it gets all of the supported formats by default. |
Even though some of us aren't quick-to-the-draw, that doesn't mean we aren't following the conversation. ;) Thanks for all your work so far! Hopefully, I'll get a chance to grab this and given it a try in the next few days. With luck that will help clarify some of these concepts that your driving at. |
For CJ I used readers and writers, parser works. I am not thrilled with just adapter because it is so generic, the adapter does have a more specific purpose. Even though it is a bit verbose I'd almost say it's a mediaTypeParser/mediaTypeWriter or maybe mediaTypeAdapter. |
@glennblock I lean in your direction w/r to how we might name these. @smizell's adapter is a little generic and serializer/deserializer is more technical than having semantic richness in the domain of these things. |
I'm not attached to the name adapter either, but used it for lack of a better term at the time. Also, just a thought, but in addition to media type adapters, there may need to be adapters for message headers. |
I forget to mention that it is great to see the progress @smizell. As for the naming, I am not super attached either, I just think adapter is so vague. Another idea is to use content, i.e. contentParser, contentGenerator as at the end of the day you are reading or writing from the content. |
I'm currently on the tail end of having a usable version of the hypermedia resource library for Python. Once it's ready, I want to work on a couple of serializer/deserializer objects. I have several questions around this.
register
in the past. I could see other names likeadd
orload
being used. Any thoughts on a name here?Thanks!
The text was updated successfully, but these errors were encountered: