Skip to content
Snippets Groups Projects

Fixed payloads for device & platform save.

All threads resolved!

Closes #85 (closed)

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Nils BrinckmannNils Brinckmann 3 years ago (Jun 22, 2021 5:31am UTC)

Merge details

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Except my little comments, everything works fine now! :thumbsup:

    By Marc Hanisch on 2021-06-21T08:00:00 (imported from GitLab)

  • Nils Brinckmann added 2 commits

    added 2 commits

    • 1c7a878e - Added default value to include the relationships (false)
    • cec798be - Changed include value based on test cases

    Compare with previous version

    By Nils Brinckmann on 2021-06-21T13:04:49 (imported from GitLab)

  • Nils Brinckmann resolved all threads

    resolved all threads

    By Nils Brinckmann on 2021-06-21T13:06:33 (imported from GitLab)

  • I added the default values. However I think it is still more clear to use the explicit value in the DeviceApi and PlatformApi classes.

    By Nils Brinckmann on 2021-06-21T13:07:23 (imported from GitLab)

  • Nils Brinckmann approved this merge request

    approved this merge request

    By Marc Hanisch on 2021-06-21T16:01:00 (imported from GitLab)

  • Nils Brinckmann mentioned in commit e87737b5

    mentioned in commit e87737b5

    By Nils Brinckmann on 2021-06-22T07:31:17 (imported from GitLab)

  • Please register or sign in to reply
    Loading