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.

Sign up for GitHub

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

Jump to bottom

Feature/multi level includes #287

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

Feature/multi level includes #287

wants to merge 4 commits into from

Commits on Aug 21, 2018

  1. test(fix): included object order is not guaranteed

    @Tronix117
    Tronix117 committed Aug 21, 2018
    Configuration menu
    Copy the full SHA
    8a48534 View commit details
    Browse the repository at this point in the history

  2. feat(serializer): support multi-level includes 

    BREAKING CHANGE: remove "requestedIncludes" option
This option should not handle the way this adapter serialize the resources, if there is embedded resources, even if they do not appear in this option, recommended behavior is to have them in the include property as well.
Security consideration: nested includes were previously not handled, it is now the case, like in the loopback documentation, you need to ensure it is secured your side.
    @Tronix117
    Tronix117 committed Aug 21, 2018
    Configuration menu
    Copy the full SHA
    201e095 View commit details
    Browse the repository at this point in the history

  3. test(nested-include): fix travis issue

    @Tronix117
    Tronix117 committed Aug 21, 2018
    Configuration menu
    Copy the full SHA
    0b3c89c View commit details
    Browse the repository at this point in the history

  4. ci(travis): node@4 is in End-Of-Life, test long term supported versions

    @Tronix117
    Tronix117 committed Aug 21, 2018
    Configuration menu
    Copy the full SHA
    dc36ee0 View commit details
    Browse the repository at this point in the history