feat: notify self-hosted users that execution/site logs are unavailable#3071
Draft
ChiragAgg5k wants to merge 1 commit into
Draft
feat: notify self-hosted users that execution/site logs are unavailable#3071ChiragAgg5k wants to merge 1 commit into
ChiragAgg5k wants to merge 1 commit into
Conversation
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.
What
Self-hosted open-source Appwrite no longer stores function/site execution log and error output. This adds an informational notice in the console so users understand why the Logs and Errors sections may appear empty, instead of just seeing "No logs found."
How
src/lib/components/logs/selfHostedLogsAlert.svelte— aproduct-aware (site|function)Alert.Inline status="info", mirroring the existingloggingAlert.sveltepattern.src/lib/components/logs/logsResponse.svelte— the shared component that renders the Logs/Errors tabs for both functions and sites. ImportsisSelfHostedfrom$lib/systemand renders the new alert in the Logs and Errors branches when on self-hosted and no output is present (taking precedence over the oldLoggingAlert/ "No logs found." fallbacks).Because the Logs/Errors rendering is shared, this single change covers both function executions (
functions/.../executions/sheet.svelte) and site logs (sites/.../logs/sheet.svelte).Behavior
isCloudusers are unaffected).Copy
Function:
Site:
Testing
bun run format— cleanbun run lint(eslint on touched files) — cleanbun run check— no new errors from touched files (pre-existing@appwrite.io/consolemodule-resolution errors are environment-only, SDK not installed locally)