The Wayback Machine - https://web.archive.org/web/20200522030315/https://github.com/axios/axios/issues/2618
Skip to content

/ axios

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

Jump to bottom

Mention different merge mechanisms for config fields #2618

Open
chinesedfan opened this issue Dec 22, 2019 · 1 comment · May be fixed by #2631
Open

Mention different merge mechanisms for config fields #2618

chinesedfan opened this issue Dec 22, 2019 · 1 comment · May be fixed by #2631
Labels
type:documentation
Projects
v0.20.0

Comments

@chinesedfan
Copy link
Collaborator

@chinesedfan chinesedfan commented Dec 22, 2019

Section/Content To Improve
Section "Config order of precedence".

Suggested Improvement
The current implementation of mergeConfig use different merge mechanisms depends on different fields. Sometimes users felt confused, like #1000.

We should mention detailed fields in the document, or at least provide a link to the source code.

Relevant File(s): README.md
This was referenced Dec 22, 2019
Closed
Closed
@leethomas leethomas linked a pull request that will close this issue Dec 30, 2019
Open
@chinesedfan chinesedfan added this to To do in v0.20.0 Jan 24, 2020
@nyitnyitnet
Copy link

@nyitnyitnet nyitnyitnet commented Jan 30, 2020

Jack tk j":
@chinesedfan chinesedfan moved this from To do to Review in progress in v0.20.0 Jan 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:documentation
3 participants
@chinesedfan @jasonsaayman @nyitnyitnet
You can’t perform that action at this time.