-
Notifications
You must be signed in to change notification settings - Fork 129
Add more context to billing and user logs #6751
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
begelundmuller
approved these changes
Feb 24, 2025
kaspersjo
added a commit
that referenced
this pull request
Feb 24, 2025
briangregoryholmes
added a commit
that referenced
this pull request
Feb 26, 2025
…6763) * fix(cloud-ui): redirect to login on expired session (#6733) * Safer redirect (homepage) * Safer redirect (CLI auth page) * Safer redirect (GitHub connect pages) * feat: public api for embeds (#6680) * feat: first rpc stub * add emit * feat: embed public api * fix: better name for state stream * fix: error handling and better spec conformatiy * fix: cleanup * fix: format * fix: make id optional * fix: first stub of tests * Revert "fix: first stub of tests" This reverts commit 2e33082. * fix: clear method names * fix: check input * fix: gitignore * fix: ignore * Revert "fix: ignore" This reverts commit 46ee2b3. * Revert "fix: gitignore" This reverts commit ec97c8e. * fix: add tests * Merge remote-tracking branch 'origin/main' into feat/emit-state-embed * fix: comments * fix: moving org and project to const * fix: moving embed to test hook * fix: format * fix: read token * fix: more fixes * fix: comments * fix: format * .gitignore * delete test file * fix: pr comments * fix: cleaner init * fix: format * fix: lint * fix: more lint (in my pocket) * fix: wait for bids * fix: format * fix: nits * fix: format * Fix accidentally skipped migration 0057.sql (#6737) * Canvas: add support for local time filters (#6673) * support individual metrics view for time controls * Add time controls for component * Create new helper method for filters * Working time filters * Match dimension filter toggle to mocks * Remove old filter references * Lint fix * Reset filters if metrics view is changed * Use local comparison range label for KPI * lint fix * Enable grid * Fix copy * Disable comparison for local filters by default * add image to concepts RD vs RC (#6729) * fix: Support CSV and Table output for `rill query` --format (#6741) * beautify csv and human output * add comments * Canvas: add stacked bar normalized (#6740) * Canvas: tweak image component to fit all image sizes (#6705) * Tweak: Do not show "New source" prompt if source has previously been ingested (#6616) * Do not show "New source" prompt if the source has already once been ingested * Add an "X" button in top-right of Alert Dialog * feat: small improvements to search (#6736) * Limit the max values in leaderboard to 15 * Add loading and error state for search * Fix lint * PR comments * fix: global properties should be set with global scope (#6746) * nit: trim semi colon in sql resolver (#6745) * trim semi colon in sql resolver * trim semi colon in sql resolver * fix: check for homedir on rill start (#6744) * fix: check for homedir on rill start * fix: random test failure * Revert "fix: random test failure" This reverts commit a8a13ec. * Use org name for autoscaler logs (#6747) * Use org name for autoscaler logs * Mark `dependency_error: true` for reconcile errors due to a dependency failure (#6732) * mark for reconcile errors due to a dependency failure * review * Update runtime/controller.go Co-authored-by: Benjamin Egelund-Müller <[email protected]> * review comments --------- Co-authored-by: Benjamin Egelund-Müller <[email protected]> * Add more context to billing and user logs (#6751) * tweak: rewrite kpi component, canvas cleanup (#6730) * wip * cleanup and layout adjustments * revert comment out * revert change * feedback * feedback * remove debug element * force ci * feat (Explore): Show the "last refreshed" date to everyone (#6750) * Integrate with new `modelRefreshedOn` property * Integrate with new `modelRefreshedOn` property (listings) * Delete unused function * Add comments * Fix null date * fix (microcopy): "source" -> "model" (#6754) * Fix microcopy: "source" -> "model" * Fix microcopy: "source" -> "model" * fix: do not select if shift key is used (#6758) * fix(explore): show error component not an infinite spinner (#6749) * fix (exports): handle filters and searches (#6742) * refactor: Push export logic into `ExportMenu` component * Rename `CreateScheduledReportDialog` -> `ScheduledReportDialog` * Handle scheduled reports * fix: Account for both filters and searches * Rename `scheduledReportsQueryArgs` -> `exportQueryArgs` * Remove wrapper function * Move fn to common location * Make function easier-to-test * Add unit tests * Self review * Fix time range for on-demand exports * Fix lint * Review * Review pt 2 * fix: Use cached plan details for page load to avoid calling orb's APIs (#6739) * Use cached plan details for page load * Replicate plan type detection in frontend * Fix lint * Fix race condition in showing billing banner * Fix all time in yaml crashing the app (#6760) * Add organization billing plan name to default deployment annotations (#6761) * init commit * cleanup * remove import * fix (Cloud UI): Always redirect to login when auth token expires (#6766) * Catch expired tokens in root `+layout.ts` * Use native SvelteKit redirect (to avoid error page flash) * e2e: Capture logs from Go services (#6771) * Save `admin` and `runtime` output to `playwright/logs/admin-runtime.log` * Upload logfile in GitHub Action * fix panic due to uri parse error (#6772) * remove log * remove log * add comment * always show outline around selected components, hide comparison line in KPI * reorg, cleanup and safari fixes * type fix --------- Co-authored-by: Eric P Green <[email protected]> Co-authored-by: Alexander Thor <[email protected]> Co-authored-by: Benjamin Egelund-Müller <[email protected]> Co-authored-by: Dhiraj Barnwal <[email protected]> Co-authored-by: Roy Endo <[email protected]> Co-authored-by: Graham Plata <[email protected]> Co-authored-by: Aditya Hegde <[email protected]> Co-authored-by: Anshul Khandelwal <[email protected]> Co-authored-by: Himadri Singh <[email protected]> Co-authored-by: Kasper Sjørslev <[email protected]>
begelundmuller
added a commit
that referenced
this pull request
Feb 27, 2025
* Implement `rows:` in the canvas YAML * Self review * Fix test * Potential fix for code scanning alert no. 31: Incorrect conversion between integer types Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> * Optional width/height * Flatten inline component definitions * Fix Go test * feat: visual canvas editor yaml structure integration and refinement (#6763) * fix(cloud-ui): redirect to login on expired session (#6733) * Safer redirect (homepage) * Safer redirect (CLI auth page) * Safer redirect (GitHub connect pages) * feat: public api for embeds (#6680) * feat: first rpc stub * add emit * feat: embed public api * fix: better name for state stream * fix: error handling and better spec conformatiy * fix: cleanup * fix: format * fix: make id optional * fix: first stub of tests * Revert "fix: first stub of tests" This reverts commit 2e33082. * fix: clear method names * fix: check input * fix: gitignore * fix: ignore * Revert "fix: ignore" This reverts commit 46ee2b3. * Revert "fix: gitignore" This reverts commit ec97c8e. * fix: add tests * Merge remote-tracking branch 'origin/main' into feat/emit-state-embed * fix: comments * fix: moving org and project to const * fix: moving embed to test hook * fix: format * fix: read token * fix: more fixes * fix: comments * fix: format * .gitignore * delete test file * fix: pr comments * fix: cleaner init * fix: format * fix: lint * fix: more lint (in my pocket) * fix: wait for bids * fix: format * fix: nits * fix: format * Fix accidentally skipped migration 0057.sql (#6737) * Canvas: add support for local time filters (#6673) * support individual metrics view for time controls * Add time controls for component * Create new helper method for filters * Working time filters * Match dimension filter toggle to mocks * Remove old filter references * Lint fix * Reset filters if metrics view is changed * Use local comparison range label for KPI * lint fix * Enable grid * Fix copy * Disable comparison for local filters by default * add image to concepts RD vs RC (#6729) * fix: Support CSV and Table output for `rill query` --format (#6741) * beautify csv and human output * add comments * Canvas: add stacked bar normalized (#6740) * Canvas: tweak image component to fit all image sizes (#6705) * Tweak: Do not show "New source" prompt if source has previously been ingested (#6616) * Do not show "New source" prompt if the source has already once been ingested * Add an "X" button in top-right of Alert Dialog * feat: small improvements to search (#6736) * Limit the max values in leaderboard to 15 * Add loading and error state for search * Fix lint * PR comments * fix: global properties should be set with global scope (#6746) * nit: trim semi colon in sql resolver (#6745) * trim semi colon in sql resolver * trim semi colon in sql resolver * fix: check for homedir on rill start (#6744) * fix: check for homedir on rill start * fix: random test failure * Revert "fix: random test failure" This reverts commit a8a13ec. * Use org name for autoscaler logs (#6747) * Use org name for autoscaler logs * Mark `dependency_error: true` for reconcile errors due to a dependency failure (#6732) * mark for reconcile errors due to a dependency failure * review * Update runtime/controller.go Co-authored-by: Benjamin Egelund-Müller <[email protected]> * review comments --------- Co-authored-by: Benjamin Egelund-Müller <[email protected]> * Add more context to billing and user logs (#6751) * tweak: rewrite kpi component, canvas cleanup (#6730) * wip * cleanup and layout adjustments * revert comment out * revert change * feedback * feedback * remove debug element * force ci * feat (Explore): Show the "last refreshed" date to everyone (#6750) * Integrate with new `modelRefreshedOn` property * Integrate with new `modelRefreshedOn` property (listings) * Delete unused function * Add comments * Fix null date * fix (microcopy): "source" -> "model" (#6754) * Fix microcopy: "source" -> "model" * Fix microcopy: "source" -> "model" * fix: do not select if shift key is used (#6758) * fix(explore): show error component not an infinite spinner (#6749) * fix (exports): handle filters and searches (#6742) * refactor: Push export logic into `ExportMenu` component * Rename `CreateScheduledReportDialog` -> `ScheduledReportDialog` * Handle scheduled reports * fix: Account for both filters and searches * Rename `scheduledReportsQueryArgs` -> `exportQueryArgs` * Remove wrapper function * Move fn to common location * Make function easier-to-test * Add unit tests * Self review * Fix time range for on-demand exports * Fix lint * Review * Review pt 2 * fix: Use cached plan details for page load to avoid calling orb's APIs (#6739) * Use cached plan details for page load * Replicate plan type detection in frontend * Fix lint * Fix race condition in showing billing banner * Fix all time in yaml crashing the app (#6760) * Add organization billing plan name to default deployment annotations (#6761) * init commit * cleanup * remove import * fix (Cloud UI): Always redirect to login when auth token expires (#6766) * Catch expired tokens in root `+layout.ts` * Use native SvelteKit redirect (to avoid error page flash) * e2e: Capture logs from Go services (#6771) * Save `admin` and `runtime` output to `playwright/logs/admin-runtime.log` * Upload logfile in GitHub Action * fix panic due to uri parse error (#6772) * remove log * remove log * add comment * always show outline around selected components, hide comparison line in KPI * reorg, cleanup and safari fixes * type fix --------- Co-authored-by: Brian Holmes <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.