Skip to content

Conversation

@Dosant
Copy link
Contributor

@Dosant Dosant commented Oct 28, 2020

Summary

Follow up on #76889
Built on top of #81297
Part of #61738

This pr add restore methods as described in RFC

I need this for:

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@Dosant Dosant added the Feature:Search Querying infrastructure in Kibana label Oct 28, 2020
@Dosant
Copy link
Contributor Author

Dosant commented Oct 29, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

merge conflict between base and head

@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes Team:AppArch v7.11.0 v8.0.0 labels Oct 29, 2020
@Dosant Dosant marked this pull request as ready for review October 29, 2020 15:57
@Dosant Dosant requested a review from a team as a code owner October 29, 2020 15:57
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@Dosant Dosant requested a review from lizozom October 29, 2020 15:57
export class SessionService implements ISessionService {
private sessionId?: string;
private session$: Subject<string | undefined> = new Subject();
private session$ = new BehaviorSubject<string | undefined>(undefined);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@lizozom lizozom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I would add a functional Discover test with this PR.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

  • 💚 Build #84353 succeeded ee475387ad2e626892d88f8ac8aa0ca44c3a6890

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@Dosant
Copy link
Contributor Author

Dosant commented Oct 30, 2020

Discussed that functional test will be added when this api will be actually used:
#81489
#81633

@Dosant Dosant merged commit aa620bc into elastic:master Oct 30, 2020
Dosant added a commit to Dosant/kibana that referenced this pull request Oct 30, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request Oct 30, 2020
* master: (71 commits)
  [Chrome] Extension to append an element to the last breadcrumb (elastic#82015)
  [Monitoring] Thread pool rejections alert (elastic#79433)
  [Actions] Fix actionType type on registerType function (elastic#82125)
  [Security Solution] Modal for saving timeline (elastic#81802)
  add tests for index pattern switching (elastic#81987)
  TS project references for share plugin (elastic#82051)
  [Graph] Fix problem with duplicate ids (elastic#82109)
  skip 'returns a single bucket if array has 1'.  related elastic#81460
  Add a link to documentation in the alerts and actions management UI (elastic#81909)
  [Fleet] fix duplicate ingest pipeline refs (elastic#82078)
  Context menu trigger for URL Drilldown (elastic#81158)
  SO management: fix legacy import index pattern selection being reset when switching page (elastic#81621)
  Fixed dead links (elastic#78696)
  [Search] Add "restore" to session service (elastic#81924)
  fix Lens heading structure (elastic#81752)
  [ML] Data Frame Analytics: Fix feature importance cell value and decision path chart (elastic#82011)
  Remove legacy app arch items from codeowners. (elastic#82084)
  [TSVB] Renamed 'positive rate' to 'counter rate' (elastic#80939)
  Expressions/migrations2 (elastic#81281)
  [Telemetry] [Schema] remove number type and support all es number types (elastic#81774)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Search Querying infrastructure in Kibana release_note:skip Skip the PR/issue when compiling release notes v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants