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

Map existing configuration of (everything) to the 6 steps #6

Open
mjtamlyn opened this issue Apr 3, 2016 · 5 comments
Open

Map existing configuration of (everything) to the 6 steps #6

mjtamlyn opened this issue Apr 3, 2016 · 5 comments
Assignees

Comments

@mjtamlyn
Copy link
Owner

mjtamlyn commented Apr 3, 2016

@ian-foote

@LilyFoote
Copy link
Collaborator

Currently working on this at https://github.com/Ian-Foote/new-serializers-notes.

@LilyFoote LilyFoote self-assigned this Apr 3, 2016
@LilyFoote
Copy link
Collaborator

Where do we want to have metadata like labels and help text? Clearly these are used in the render step, but they're also the sort of thing that might appear in a DRF Metadata class to be returned by an OPTIONS request.

@mjtamlyn
Copy link
Owner Author

mjtamlyn commented Apr 3, 2016

They are part of the "rendering" step, but that doesn't mean that they only belong in the renderer. Thinking about the structural object underneath, it looking something like:

{
    'fields': [...],
    'serialization': [...],
    'input': [...],
    'deserialization': [...],
    'rendering': [...],
}

or something along those lines might make sense. fields might just be a list of names which everything else refers to. It's entirely possible that the API for declaring things may flatten this stuff together, but we can extract it like the above. Rendering step could optionally include layout information for example if you like that kind of thing.

@LilyFoote
Copy link
Collaborator

I think I've covered most of the public Django forms api now. On to DRF!

@LilyFoote
Copy link
Collaborator

I've added some notes on Serializer and its subclasses: LilyFoote/new-serializers-notes@92c4668

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

No branches or pull requests

2 participants