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

Cleanup aiAssistantFlyoutMode feature flag #182992

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

patrykkopycinski
Copy link
Contributor

@patrykkopycinski patrykkopycinski commented May 8, 2024

Summary

Cleanup Security aiAssistantFlyoutMode feature flag

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

…-modal-cleanup

# Conflicts:
#	x-pack/plugins/security_solution/public/attack_discovery/pages/header/index.tsx
@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski patrykkopycinski changed the title Chore/assistant modal cleanup Cleanup aiAssistantFlyoutMode feature flag May 9, 2024
@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

…-modal-cleanup

# Conflicts:
#	x-pack/packages/kbn-elastic-assistant/impl/assistant/assistant_header/assistant_header_flyout.tsx
#	x-pack/packages/kbn-elastic-assistant/impl/assistant/index.tsx
@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@patrykkopycinski
Copy link
Contributor Author

/ci

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 5482 5475 -7

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/elastic-assistant 146 145 -1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 15.1MB 15.0MB -123.8KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
securitySolution 83.5KB 83.5KB -25.0B
Unknown metric groups

API count

id before after diff
@kbn/elastic-assistant 173 172 -1

async chunk count

id before after diff
securitySolution 97 96 -1

ESLint disabled in files

id before after diff
@kbn/elastic-assistant 2 1 -1

ESLint disabled line counts

id before after diff
@kbn/elastic-assistant 17 13 -4
securitySolution 522 521 -1
total -5

Total ESLint disabled count

id before after diff
@kbn/elastic-assistant 19 14 -5
securitySolution 604 603 -1
total -6

History

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

@patrykkopycinski patrykkopycinski marked this pull request as ready for review May 17, 2024 11:57
@patrykkopycinski patrykkopycinski requested review from a team as code owners May 17, 2024 11:57
@patrykkopycinski patrykkopycinski self-assigned this May 17, 2024
@patrykkopycinski patrykkopycinski added release_note:skip Skip the PR/issue when compiling release notes v8.15.0 labels May 17, 2024
@@ -218,14 +213,10 @@ describe('Assistant', () => {
);
});

it('should delete conversation when delete button is clicked', async () => {
it.skip('should delete conversation when delete button is clicked', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

supposed to be skipped?

@@ -235,7 +226,7 @@ describe('Assistant', () => {
expect(mockDeleteConvo).toHaveBeenCalledWith(mockData.welcome_id.id);
});
});
describe('when selected conversation changes and some connectors are loaded', () => {
describe.skip('when selected conversation changes and some connectors are loaded', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

skipped?

Copy link
Contributor

@PhilippeOberti PhilippeOberti left a comment

Choose a reason for hiding this comment

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

desk tested and code LGTM for the Threat Hunting Investigations team!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants