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

Add audit log documentation #439

Merged
merged 8 commits into from
Nov 27, 2024
Merged

Add audit log documentation #439

merged 8 commits into from
Nov 27, 2024

Conversation

georgegevoian
Copy link
Collaborator

No description provided.

Copy link

netlify bot commented Nov 3, 2024

Deploy Preview for grist-help-preview ready!

Name Link
🔨 Latest commit 8c7d3b8
🔍 Latest deploy log https://app.netlify.com/sites/grist-help-preview/deploys/6743bfa2a43d990008a7efc4
😎 Deploy Preview https://deploy-preview-439--grist-help-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@georgegevoian georgegevoian marked this pull request as ready for review November 15, 2024 09:02
help/en/docs/install/audit-log-overview.md Outdated Show resolved Hide resolved
help/en/docs/install/audit-log-overview.md Outdated Show resolved Hide resolved
help/en/docs/install/audit-log-overview.md Outdated Show resolved Hide resolved
help/en/docs/install/audit-log-overview.md Show resolved Hide resolved
help/en/docs/install/audit-log-overview.md Outdated Show resolved Hide resolved
Events are streamed in real-time. Only events that occurred after a streaming
destination was added will be sent to that destination.

**Note:** You must be an installation admin to view or modify streaming
Copy link
Member

Choose a reason for hiding this comment

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

*administrator

I wonder if we explain what an installation administrator is anywhere, or how to know if you are one, or how to become one. For grist-core I think it is very simple, there's only one, set by that silly default email env variable. For SaaS or maybe enterprise we have an extra mechanism, a special site whose members are admins, right? Ah, just checked, that's only on our SaaS currently, not in enterprise. If in enterprise it could have been worth documenting. (@dsagal I wonder if the GRIST_INSTALL_ADMIN_ORG mechanism would be worth moving?)

Ok, for auditing, fine not to get into it.

help/en/docs/install/audit-log-streaming.md Outdated Show resolved Hide resolved
help/en/docs/install/audit-log-streaming.md Outdated Show resolved Hide resolved
help/en/docs/install/audit-log-streaming.md Show resolved Hide resolved
help/en/docs/install/audit-log-events.md Show resolved Hide resolved
Copy link
Member

@paulfitz paulfitz left a comment

Choose a reason for hiding this comment

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

Looks good to merge when the feature is ready.

@nbush nbush merged commit f1897e6 into master Nov 27, 2024
5 checks passed
@nbush nbush deleted the audit-logs branch November 27, 2024 16:05
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