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

Correct external application properties load order #24308

Closed
philwebb opened this issue Dec 1, 2020 · 1 comment
Closed

Correct external application properties load order #24308

philwebb opened this issue Dec 1, 2020 · 1 comment
Assignees
Labels
status: noteworthy A noteworthy issue to call out in the release notes type: bug A general bug
Milestone

Comments

@philwebb
Copy link
Member

philwebb commented Dec 1, 2020

Originally raised in #24211, we'd like to change the order in the code rather than the docs.

@philwebb philwebb added this to the 2.4.x milestone Dec 1, 2020
@philwebb philwebb added the type: bug A general bug label Dec 1, 2020
@MikeN123
Copy link

MikeN123 commented Dec 1, 2020

Note that the docs would still require a change as well in this case. I can't figure out how to get the source of the asciidoc to link to in Github now, but in this list in the docs the order is presented as it is in the code at the moment:

Screenshot 2020-12-01 at 17 55 28

@mbhave mbhave self-assigned this Dec 3, 2020
@mbhave mbhave added the status: noteworthy A noteworthy issue to call out in the release notes label Dec 3, 2020
@mbhave mbhave closed this as completed in 932fb65 Dec 3, 2020
@mbhave mbhave modified the milestones: 2.4.x, 2.4.1 Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: noteworthy A noteworthy issue to call out in the release notes type: bug A general bug
Projects
None yet
Development

No branches or pull requests

3 participants