Skip to content

Conversation

@amee-sumo
Copy link
Collaborator

@amee-sumo amee-sumo commented Feb 6, 2025

Purpose of this pull request

This pull request is to update 9 apps with the new app+C2C flow.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/DOCS-672

@amee-sumo amee-sumo added doc:update Revisions and updates to content do-not-merge Pull requests that should not be merged labels Feb 6, 2025
@amee-sumo amee-sumo self-assigned this Feb 6, 2025
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Feb 6, 2025
Copy link
Collaborator

@JV0812 JV0812 left a comment

Choose a reason for hiding this comment

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

Minor fixes, also resize the dashboard images for Google BigQuery app.

amee-sumo and others added 6 commits February 6, 2025 15:04
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
@jpipkin1
Copy link
Collaborator

jpipkin1 commented Feb 6, 2025

@amee-sumo, this PR was failing the build-and-deploy check, so I submitted commit Fix build error to fix it. The problem was that the definitions for the tabs have to match exactly, including casing.

@amee-sumo amee-sumo requested a review from JV0812 February 11, 2025 06:30
@amee-sumo
Copy link
Collaborator Author

@amee-sumo, this PR was failing the build-and-deploy check, so I submitted commit Fix build error to fix it. The problem was that the definitions for the tabs have to match exactly, including casing.

Thanks for the changes @jpipkin1 .. will keep that in mind.

Copy link
Collaborator

@JV0812 JV0812 left a comment

Choose a reason for hiding this comment

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

LGTM

amee-sumo and others added 5 commits February 11, 2025 14:00
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
Co-authored-by: Jagadisha V <129049263+JV0812@users.noreply.github.com>
@amee-sumo amee-sumo requested a review from JV0812 February 11, 2025 08:31
@amee-sumo amee-sumo enabled auto-merge February 11, 2025 08:32
@amee-sumo amee-sumo added this pull request to the merge queue Feb 11, 2025
Merged via the queue into main with commit 47f2e00 Feb 11, 2025
7 checks passed
@amee-sumo amee-sumo deleted the app+c2c-installation-flow-update-(Apps) branch February 11, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged doc:update Revisions and updates to content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants