This PR fixes a bug where the error was not displaying when misconfigured connector was connected and redirected back. The issue was that the redirect was occuring after the flash message was set and the fix was to redirect and then show the flash message
* Fix error appearing if user leaves Sources page very quickly
The issue was that the response from /sources endpoint could came after the user has left the page.
Since the user has already left the page, the Sources logic is unmounted,
and any code that was using the response couldn't update the value in that logic file and caused an error.
Fortunately Kea provides a `breakpoint` API exactly for such cases:
https://kea.js.org/docs/guide/additional#breakpoints
This commit uses that API to fix the issue.
* Fix error appearing after leaving Sources page less quickly
This commit solves the same problem, but for the /status endpoint:
1) for the first status call that saves server values to the sources_logic
2) for the subsequent status calls that poll the server for the status updates
* Increase test coverage
The new test duplicates the test below it, but it doesn't set up
initial source statuses. For some reason this case was considered to be covered before,
but after seemingly unrelated changes in this PR, the coverage started to show that this
code branch was missed.
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
* Add attachment to list of synced objects for Kibana
* Add attachments to Salesforce Sandbox as well
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
There is an edge case where a connector does not sync and there is no estimate of next syncs in the UI. In this case, we simply omit the estimates from the UI.
* Add icons
* Add new categories
* Add Workplace Search connectors to the unified integrations view
* Add a new enterprise_search shipper
* Update number of custom integrations in test
* Change customIntegrations type to optional
* Revert "Update number of custom integrations in test"
This reverts commit 30214b2c7c.
The reason is that while this test passes with 2 separate commands for
functional test runner, it fails when it is run with a single command.
Reverting to make the CI pass. We will look into this test separately.
I will link the investigation issue in the PR.
* Change source deletion to set entire page loading
Do this instead of having the button loading
* Replace text button with icon button for blocked windows
* Remove extra source title from Personal dashboard
* Change button order to match other views
We typically have the right-most button the Save button and the reset button to the left
* Fix typo
* Fix failing test
EUI requires the name but we don’t want to dispaly it, so sending an empty string
* Remove Synchronization nav items from Custom Source
* Hide syncTriggerCallout for custom sources