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

support gemini-usage connector #858

Open
wants to merge 9 commits into
base: rc-v0.5.1
Choose a base branch
from
Open

Conversation

eschultink
Copy link
Member

Features

  • An initial version of the gemini-usage connector, which includes documentation for setup and usage

Change implications

  • dependencies added/changed? no
  • something important to note in future release notes? added Gemini Usage connector

@eschultink eschultink self-assigned this Mar 5, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR adds an initial version of the Gemini Usage connector documentation to guide users through setup and usage.

  • Added a new README file with details about downloading and processing Gemini Usage reports.
  • Provided step‐by‐step instructions to configure the connector via Terraform.

Reviewed Changes

File Description
docs/sources/google-workspace/gemini-usage/README.md New documentation for the Gemini Usage connector detailing setup and usage instructions

Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.

Comment on lines 39 to 40
5. Create the a `gemini-usage` connection in Worklytics; see `TODO 3 - gemini-usage` file that your `terraform apply` generated.

Copy link
Preview

Copilot AI Mar 5, 2025

Choose a reason for hiding this comment

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

[nitpick] There is a duplicate article 'the a' in the instruction. Please modify it to either 'Create a gemini-usage connection...' or 'Create the gemini-usage connection...'.

Suggested change
5. Create the a `gemini-usage` connection in Worklytics; see `TODO 3 - gemini-usage` file that your `terraform apply` generated.
5. Create a `gemini-usage` connection in Worklytics; see `TODO 3 - gemini-usage` file that your `terraform apply` generated.

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
@@ -214,6 +214,7 @@ module "bulk_connector" {
bucket_write_role_id = module.psoxy.bucket_write_role_id
secret_bindings = module.psoxy.secrets
example_file = try(each.value.example_file, null)
instructions_template = try(each.value.instructions_template, null)
Copy link
Contributor

Choose a reason for hiding this comment

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

Commented here, but applies to other files: indent?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, we could move rest of connector instructions with templates like this so we ensure all docs are updated (and no need to keep tf and md versions)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants