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

docs(usage-view): how to use csv data export #438

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

mwasilew2
Copy link
Contributor

@mwasilew2 mwasilew2 commented Jan 24, 2024

Description of the change

related to: https://app.clickup.com/t/8693k73r6

Adds docs about downloading and analyzing usage data.

Checklist

Please make sure that the proposed change checks all the boxes below before requesting a review:

  • I have reviewed the guidelines for contributing to this repository.
  • The preview looks fine.
  • The tests pass.
  • The commit history is clean and meaningful.
  • The pull request is opened against the main branch.
  • The pull request is no longer marked as a draft.
  • You agree to license your contribution under the MIT license to Spacelift (not required for Spacelift employees).
  • You have updated the navigation files correctly:
    • No new pages have been added, or;
    • Only nav.yaml has been updated because the changes only apply to SaaS, or;
    • Only nav.self-hosted.yaml has been updated because the changes only apply to Self-Hosted, or;
    • Both nav.yaml and nav.self-hosted.yaml have been updated.

If the proposed change is ready to be merged, please request a review from @spacelift-io/solutions-engineering. Someone will review and merge the pull request.

Spacelift employees should request reviews from the relevant engineers and are allowed to merge pull requests after they got at least one approval.

Thank you for your contribution! 🙇

Copy link

render bot commented Jan 24, 2024

@mwasilew2 mwasilew2 marked this pull request as ready for review January 25, 2024 16:39
@mwasilew2
Copy link
Contributor Author

@flavius-dinu I tagged you cause I saw you did some work on docs before. Let me know if there's someone else I should be asking for review. Thanks!

Copy link
Contributor

@jakubdal jakubdal left a comment

Choose a reason for hiding this comment

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

Please wait for run state translation and recreate the image. Also please wait for feature GA before publishing. LGTM overall.

@mwasilew2 mwasilew2 force-pushed the CU-8693k73r6-usage-view-csv-docs branch from 21579b0 to 1f2d13e Compare January 29, 2024 14:24
KNiepok
KNiepok previously approved these changes Jan 30, 2024
@mwasilew2
Copy link
Contributor Author

this is ready to be merged when the feature goes GA

Signed-off-by: Michal Wasilewski <michal@mwasilewski.net>
@mwasilew2 mwasilew2 force-pushed the CU-8693k73r6-usage-view-csv-docs branch from 0856cd5 to 326c6a9 Compare February 16, 2024 12:32
@mwasilew2 mwasilew2 merged commit 5cbf8ad into main Feb 16, 2024
1 check passed
@mwasilew2 mwasilew2 deleted the CU-8693k73r6-usage-view-csv-docs branch February 16, 2024 12:46
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.

3 participants