Skip to content

Merge configs #97

Description

@innovate-invent

Currently object unpacking is being used to combine all of the configs. This does not allow setting axios settings higher up in the config stack that are objects.
For example, if I want a global param for all requests I would expect to be able to go:

VuexORM.use(VuexORMAxios, {
	params: {
		key: "foo"
	}
});

Currently any lower config specifying params will overwrite this.
Something along the lines of lodash merge should be used.
If I wanted to unset a setting higher up I can simply do something like:

Model.get('url', {
	params: {
		key: null
	}
});

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions