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

Implement error object links. #139

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Implement error object links. #139

wants to merge 17 commits into from

Commits on Jul 27, 2017

  1. Embedded structs v3 (#100)

    * working version
    
    * fix text
    
    * combine test files
    
    * move private funcs to bottom
    
    * ErrInvalidType should ignore interfaces
    
    * replace MarshalOnePayload w/ MarshalPayload; fix bug w/ node merge()
    
    * minor tweaks; address a couple comments
    
    * decompose unmarshalNode() to smaller funcs; unmarshal should go from top-level to embedded
    
    * deep copy the node when passing relation/sideloaded notes to unmarshal()
    
    * add some comments and do some additional cleanup
    
    * add test uses annotationIgnore
    
    * implement support for struct fields that implement json.Marshaler/Unmarshaler
    
    * add additional test that compares marshal/unmarshal behavior w/ standard json library
    
    * add support for pointer embedded structs
    
    * Revert "implement support for struct fields that implement json.Marshaler/Unmarshaler"
    
    This reverts commit deeffb7.
    skimata authored and aren55555 committed Jul 27, 2017
    Configuration menu
    Copy the full SHA
    a6ac768 View commit details
    Browse the repository at this point in the history
  2. Added a failing test for the allocation of struct pointers on seriali…

    …zation/deserilization.
    aren55555 committed Jul 27, 2017
    Configuration menu
    Copy the full SHA
    3e612cc View commit details
    Browse the repository at this point in the history

Commits on Jul 28, 2017

  1. Added another failing test; this time for the marshalling of duplicat…

    …e primary annotation fields.
    aren55555 committed Jul 28, 2017
    Configuration menu
    Copy the full SHA
    003d45f View commit details
    Browse the repository at this point in the history
  2. Test against 1.9

    aren55555 committed Jul 28, 2017
    Configuration menu
    Copy the full SHA
    5be0508 View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2017

  1. Feature/embeded structs fix tests (#105)

    * fix TestEmbededStructs_nonNilStructPtr; bug on loop w/ (multiple) embedded structs; should just continue on success
    
    * fix TestMarshal_duplicatePrimaryAnnotationFromEmbeddedStructs; fix order of processing of visitModelNode
    skimata authored and aren55555 committed Aug 1, 2017
    Configuration menu
    Copy the full SHA
    662431e View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    20c2b22 View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2017

  1. Moved all embeded struct tests into its own test file; added 3 more f…

    …ailing tests that are related to the handling of duplicate embedded struct fields.
    aren55555 committed Aug 2, 2017
    Configuration menu
    Copy the full SHA
    cf9619a View commit details
    Browse the repository at this point in the history
  2. Addressing warnings.

    aren55555 committed Aug 2, 2017
    Configuration menu
    Copy the full SHA
    d94776e View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    a92a3b7 View commit details
    Browse the repository at this point in the history
  4. Misc cleanup.

    aren55555 committed Aug 2, 2017
    Configuration menu
    Copy the full SHA
    cab68da View commit details
    Browse the repository at this point in the history
  5. Added two additional tests (currently passing) that test the cases wh…

    …ere an Outer struct had an attr field and an embedded field provided a field with the same attr name.
    aren55555 committed Aug 2, 2017
    Configuration menu
    Copy the full SHA
    8e4b261 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    0ec2f4e View commit details
    Browse the repository at this point in the history

Commits on Sep 5, 2017

  1. Configuration menu
    Copy the full SHA
    7822e6f View commit details
    Browse the repository at this point in the history

Commits on May 1, 2018

  1. Configuration menu
    Copy the full SHA
    66e21ca View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    466da34 View commit details
    Browse the repository at this point in the history

Commits on Oct 22, 2018

  1. Configuration menu
    Copy the full SHA
    28f3cac View commit details
    Browse the repository at this point in the history
  2. Merge pull request #1 from jsignanini/feature/embeded-structs

    Feature/embeded structs
    jsignanini authored Oct 22, 2018
    Configuration menu
    Copy the full SHA
    731dc79 View commit details
    Browse the repository at this point in the history