Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Submit feedback
  • Sign in
chocapix-server
chocapix-server
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 5
    • Issues 5
    • List
    • Boards
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Camille MASSET
  • chocapix-serverchocapix-server
  • Issues
  • #9

Closed
Open
Opened Nov 24, 2019 by Camille MASSET@camille.masset
  • Report abuse
  • New issue
Report abuse New issue

Refactor serializers to handle default value for `bar`

With django-rest-framework 3.8, serializers fields with "default" and "read_only" behave differently from before. As a result, we have to add a function save() for each serializer that has a field:

bar = serializers.PrimaryKeyRelatedField(read_only=True, default=CurrentBarCreateOnlyDefault())

so that the field is correctly populated.

Suggested approach: create a mixin CurrentBarOnCreateMixin that implements that logic.

Edited Nov 24, 2019 by Camille MASSET
  • Discussion
  • Designs

The one place for your designs

To enable design management, you'll need to meet the requirements. If you need help, reach out to our support team for assistance.

Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
0
Labels
None
Assign labels
  • View project labels
Reference: camille.masset/chocapix-server#9