Remove browser basePath service, move functionality into browser http service #36611
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eliperelman
added
Team:Core
|
Pinging @elastic/kibana-platform |
eliperelman
force-pushed
the
basepath-to-http
branch
2 times, most recently
from
bf2f7a9 to
ffe8c34
Compare
joshdover
previously requested changes
May 17, 2019
eliperelman
force-pushed
the
basepath-to-http
branch
from
ffe8c34 to
a312ea2
Compare
|
@joshdover made changes from your review in a312ea2. |
mshustov
reviewed
May 17, 2019
src/core/public/application/capabilities/capabilities_service.tsx
Outdated
Show resolved
Hide resolved
src/core/public/application/capabilities/capabilities_service.tsx
Outdated
Show resolved
Hide resolved
src/core/public/application/capabilities/capabilities_service.test.ts
Outdated
Show resolved
Hide resolved
joshdover
dismissed
their stale review
Haven't re-reviewed yet, but it looks like @restrry has spotted some things I noticed too
eliperelman
force-pushed
the
basepath-to-http
branch
from
a312ea2 to
913b744
Compare
|
Current commits needing review have been updated in the description. |
eliperelman
force-pushed
the
basepath-to-http
branch
3 times, most recently
from
d45afbd to
8f036b5
Compare
💚 Build Succeeded |
eliperelman
force-pushed
the
basepath-to-http
branch
from
cb11b0d to
7244e40
Compare
joshdover
reviewed
May 22, 2019
joshdover
approved these changes
May 22, 2019
eliperelman
force-pushed
the
basepath-to-http
branch
from
b4e736b to
26f97bf
Compare
💔 Build Failed |
joshdover
approved these changes
May 22, 2019
💔 Build Failed |
… to removeBasePath
💚 Build Succeeded |
eliperelman
force-pushed
the
basepath-to-http
branch
from
26f97bf to
c4dab47
Compare
💔 Build Failed |
💚 Build Succeeded |
7 tasks
eliperelman
added a commit
to eliperelman/kibana
that referenced
this pull request
May 31, 2019
… service (elastic#36611) * Remove browser basePath service, move functionality into browser http service * Update generated documentation for removal of browser basePath * Fix type interface for removal of basePath * Split IHttpService into separate setup and start interfaces * Rename appendToBasePath to prependBasePath, rename removeFromBasePath to removeBasePath
eliperelman
added a commit
that referenced
this pull request
May 31, 2019
…r http service (#36611) (#37387) * Remove browser basePath service, move functionality into browser http service (#36611) * Remove browser basePath service, move functionality into browser http service * Update generated documentation for removal of browser basePath * Fix type interface for removal of basePath * Split IHttpService into separate setup and start interfaces * Rename appendToBasePath to prependBasePath, rename removeFromBasePath to removeBasePath * Fix import of http test setup
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
The browser
basePathservice is a pretty small service which only has functionality for the manipulation of Kibana'sbasePathin order to affect HTTP requests. This patch removes the browserbasePathservice in favor of handling by the browser HTTP service.Commits needing review
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.This was checked for cross-browser compatibility, including a check against IE11Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n supportDocumentation was added for features that require explanation or tutorialsThis was checked for [keyboard-only and screenreader accessibility](https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Cross_browser_testing/Accessibility#Accessibility_testing_checklistFor maintainers
Dev Docs
The browser
basePathservice is removed in favor of handling by the browserhttpservice. As thebasePathwas used for controlling the route used to make fetch requests, it makes sense to colocate this functionality into the HTTP service.