-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Uptime] Certificates page #64059
[Uptime] Certificates page #64059
Conversation
Pinging @elastic/uptime (Team:uptime) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
x-pack/legacy/plugins/uptime/public/components/certificates/cert_status.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/certificates/certificates_list.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/certificates/certificates_list.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/certificates/translations.ts
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/certificates/translations.ts
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/overview/monitor_list/cert_status_column.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/overview/monitor_list/cert_status_column.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/overview/monitor_list/monitor_list.tsx
Outdated
Show resolved
Hide resolved
x-pack/legacy/plugins/uptime/public/components/overview/monitor_list/monitor_list.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs some minor changes. Overall, very clean code!
x-pack/plugins/uptime/public/components/certificates/__tests__/cert_status.test.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/uptime/public/components/certificates/certificates_list.tsx
Outdated
Show resolved
Hide resolved
...time/public/components/monitor/monitor_status_details/monitor_status_bar/ssl_certificate.tsx
Outdated
Show resolved
Hide resolved
...ns/uptime/public/components/monitor/monitor_status_details/monitor_status_bar/status_bar.tsx
Outdated
Show resolved
Hide resolved
x-pack/test/functional/es_archives/uptime/full_heartbeat/mappings.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great feature addition, and the code is super clean and well thought-out.
I had a few comments but great that we're this close on an initial review for such a large change!
x-pack/plugins/uptime/public/components/certificates/cert_status.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/uptime/public/components/certificates/cert_status.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/uptime/public/components/certificates/certificates_list.tsx
Show resolved
Hide resolved
x-pack/plugins/uptime/public/components/certificates/certificates_list.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/uptime/public/components/certificates/fingerprint_col.tsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM Pending @justinkambic 's changes
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* upstream/master: (315 commits) [APM] Fix failing `ApmIndices` test (elastic#64965) [APM] Fix paths for ts optimization script (elastic#65012) Use HDR for percentiles (elastic#64758) [EPM] fix updates available filter (elastic#64957) [Uptime] Certificates page (elastic#64059) load lens app lazily (elastic#64769) [legacy/server/config] remove unnecessary deps for simple helper (elastic#64954) Fixed alert Edit flyout shows the error message when one of this actions has a preconfigured action type (elastic#64742) [data.search.aggs] Remove legacy aggs APIs. (elastic#64719) Fixed `AddAlert` flyout does not immediately update state to reflect new props (elastic#64927) [Discover] Show doc viewer action buttons on focus (elastic#64912) [EPM] restrict package install endpoint from installing/updating to old packages (elastic#64932) [Metrics UI] Add inventory metric threshold alerts (elastic#64292) [Canvas] Adds edit menu (elastic#64738) [Canvas] Adds refresh and autoplay options to view menu (elastic#64375) [Lens] Trigger a filter action on click in datatable visualization (elastic#63840) [SIEM][CASE] Refactor Connectors - Jira Connector (elastic#63450) [APM] Client new platform migration (elastic#64046) [Monitoring] NP Migration complete client cutover (elastic#62908) Ingest Node Pipelines UI (elastic#62321) ...
…or-part-mvp-2 * 'master' of github.com:elastic/kibana: (51 commits) [APM] Fix failing `ApmIndices` test (elastic#64965) [APM] Fix paths for ts optimization script (elastic#65012) Use HDR for percentiles (elastic#64758) [EPM] fix updates available filter (elastic#64957) [Uptime] Certificates page (elastic#64059) load lens app lazily (elastic#64769) [legacy/server/config] remove unnecessary deps for simple helper (elastic#64954) Fixed alert Edit flyout shows the error message when one of this actions has a preconfigured action type (elastic#64742) [data.search.aggs] Remove legacy aggs APIs. (elastic#64719) Fixed `AddAlert` flyout does not immediately update state to reflect new props (elastic#64927) [Discover] Show doc viewer action buttons on focus (elastic#64912) [EPM] restrict package install endpoint from installing/updating to old packages (elastic#64932) [Metrics UI] Add inventory metric threshold alerts (elastic#64292) [Canvas] Adds edit menu (elastic#64738) [Canvas] Adds refresh and autoplay options to view menu (elastic#64375) [Lens] Trigger a filter action on click in datatable visualization (elastic#63840) [SIEM][CASE] Refactor Connectors - Jira Connector (elastic#63450) [APM] Client new platform migration (elastic#64046) [Monitoring] NP Migration complete client cutover (elastic#62908) Ingest Node Pipelines UI (elastic#62321) ... # Conflicts: # x-pack/plugins/ingest_pipelines/common/types.ts # x-pack/plugins/ingest_pipelines/public/application/components/pipeline_form/pipeline_form.tsx # x-pack/plugins/ingest_pipelines/public/application/components/pipeline_form/pipeline_form_fields.tsx # x-pack/plugins/ingest_pipelines/public/shared_imports.ts
* master: (44 commits) onEvent prop for expression component (elastic#64995) [APM] Fix failing `ApmIndices` test (elastic#64965) [APM] Fix paths for ts optimization script (elastic#65012) Use HDR for percentiles (elastic#64758) [EPM] fix updates available filter (elastic#64957) [Uptime] Certificates page (elastic#64059) load lens app lazily (elastic#64769) [legacy/server/config] remove unnecessary deps for simple helper (elastic#64954) Fixed alert Edit flyout shows the error message when one of this actions has a preconfigured action type (elastic#64742) [data.search.aggs] Remove legacy aggs APIs. (elastic#64719) Fixed `AddAlert` flyout does not immediately update state to reflect new props (elastic#64927) [Discover] Show doc viewer action buttons on focus (elastic#64912) [EPM] restrict package install endpoint from installing/updating to old packages (elastic#64932) [Metrics UI] Add inventory metric threshold alerts (elastic#64292) [Canvas] Adds edit menu (elastic#64738) [Canvas] Adds refresh and autoplay options to view menu (elastic#64375) [Lens] Trigger a filter action on click in datatable visualization (elastic#63840) [SIEM][CASE] Refactor Connectors - Jira Connector (elastic#63450) [APM] Client new platform migration (elastic#64046) [Monitoring] NP Migration complete client cutover (elastic#62908) ...
Summary
Fixes: elastic/uptime#161
Added Cert Page