Skip to content

[Google Workspace] Convert visualisations to Lens - #6914

Merged
chemamartinez merged 4 commits into
elastic:mainfrom
chemamartinez:6787-google_workspace
Jul 13, 2023
Merged

[Google Workspace] Convert visualisations to Lens#6914
chemamartinez merged 4 commits into
elastic:mainfrom
chemamartinez:6787-google_workspace

Conversation

@chemamartinez

@chemamartinez chemamartinez commented Jul 11, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Updated the visualizations of Google Workspace dashboards to Lens
  • The screenshots were not updated since the integration already contains 7 screenshots and converted visualisations don't appear in them

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Related issues

Screenshots

Before
old-rules
Screenshot 2023-07-11 at 12 57 49

After
new-rules
Screenshot 2023-07-12 at 13 54 55
@chemamartinez chemamartinez self-assigned this Jul 11, 2023
@chemamartinez
chemamartinez marked this pull request as ready for review July 11, 2023 11:08
@chemamartinez
chemamartinez requested a review from a team as a code owner July 11, 2023 11:08
@elasticmachine

elasticmachine commented Jul 11, 2023

Copy link
Copy Markdown

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)
@chemamartinez chemamartinez mentioned this pull request Jul 11, 2023
Closed
36 tasks
@elasticmachine

elasticmachine commented Jul 11, 2023 •
edited
Loading

Copy link
Copy Markdown

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-07-12T11:54:33.768+0000

  • Duration: 27 min 43 sec

Test stats 🧪

Test Results
Failed 0
Passed 112
Skipped 0
Total 112

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.
@elasticmachine

elasticmachine commented Jul 11, 2023 •
edited
Loading

Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (14/14) 💚
Files 100.0% (14/14) 💚
Classes 100.0% (14/14) 💚
Methods 100.0% (179/179) 💚 0.261
Lines 94.469% (4560/4827) 👎 -1.197
Conditionals 100.0% (0/0) 💚
kcreddy
kcreddy reviewed Jul 12, 2023
View reviewed changes

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In the last visualisation (under Screenshots), the Y-Axis title Country Name doesn't seem to be accurate for what the metric is i.e., 0-1
@chemamartinez

chemamartinez commented Jul 12, 2023

Copy link
Copy Markdown
Contributor Author

In the last visualisation (under Screenshots), the Y-Axis title Country Name doesn't seem to be accurate for what the metric is i.e., 0-1

Sure thing, I've removed the Y-Axis title as it was before.
@chemamartinez
chemamartinez requested a review from kcreddy July 12, 2023 11:56
kcreddy
kcreddy approved these changes Jul 13, 2023
View reviewed changes

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
@chemamartinez
chemamartinez merged commit 43d3ba0 into elastic:main Jul 13, 2023
@elasticmachine

elasticmachine commented Jul 13, 2023

Copy link
Copy Markdown

Package google_workspace - 2.11.0 containing this change is available at https://epr.elastic.co/search?package=google_workspace
@chemamartinez
chemamartinez deleted the 6787-google_workspace branch February 6, 2025 10:30
orestisfl pushed a commit to orestisfl/integrations that referenced this pull request May 15, 2026
@chemamartinez
[Google Workspace] Convert visualisations to Lens (elastic#6914)
2bce156 
* Convert Google Workspace visualisations to Lens

* Bump integration version

* Update changelog

* Remove y-axis label in visualisation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:google_workspace Google Workspace

3 participants

@chemamartinez @elasticmachine @kcreddy