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

feat: adapterOptions #510

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

Commits on Jul 7, 2021

  1. fix: url with query params

    This PR will correctly construct a URL where the ember-api-action path
    contains query params and/or where the ember data adapter buildURL
    returns a url containing query params.
    
    Fixes mike-north#414
    mrloop committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    c551166 View commit details
    Browse the repository at this point in the history
  2. feat: adapterOptions

    Allows api actions to take an optional `options` object. If you pass an
    object on the adapterOptions property of the options argument it will be
    passed to your adapter via the snapshot.
    
    Similar to ember data Store and Modal functions, for example
    https://api.emberjs.com/ember-data/3.25/classes/Store/methods/findAll?anchor=findAll
    https://api.emberjs.com/ember-data/3.25/classes/Model/methods/save?anchor=save
    
    This is similar to mike-north#214 with tests.
    
    Closes mike-north#214
    mrloop committed Jul 7, 2021
    Configuration menu
    Copy the full SHA
    09e647d View commit details
    Browse the repository at this point in the history