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 explorer app #45

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

prashantmital
Copy link
Contributor

@prashantmital prashantmital commented Mar 28, 2023

Copy link
Contributor

@evanRetool evanRetool left a comment

Choose a reason for hiding this comment

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

The app looks great! Just have some requested additions to the README. Thanks for submitting this!


### 0. Make sure you have access!

You should have the right approvals and know where to look to find the `POSTGRES_HOST`, `POSTGRES_PORT`, `POSTGRES_DB`, `POSTGRES_USER`, `POSTGRES_PASSWORD` environment variables in your deployment configuration.
Copy link
Contributor

Choose a reason for hiding this comment

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

What should the user do if they don't know where to look to find these environment variables?

1. Navigate to the `/resources` page in your instance and click "Create a new resource"
2. Select Postgres as the resource type
3. Name the resource “Retool Audit Log”
4. To fill out the following details, depending on your deployment, check your secrets file in your deployment configuration. The relevant environment variables are:
Copy link
Contributor

Choose a reason for hiding this comment

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

Where might the user find the secrets file?

If the Usage Analytics app doesn't populate with data, it's likely you'll need to update the Resource for each query.

To update the queries to use the correct Resource, load the Usage Analytics app in editor mode and open the bottom panel to view the queries. Edit each query and select your Retool Postgres database from the Resource dropdown. (Don't forget to save your queries after editing them!)

Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add a section on how the user might start using the app? Are there some common operations they should start with? Also, is there any information we can provide about how to interpret the audit log events?

@@ -0,0 +1,78 @@
# Importable Audit Log Explorer App
Copy link
Contributor

Choose a reason for hiding this comment

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

I see a warning in the app about not running large queries due to the potential impact on other users of the instance. I think we should call that out here prominently since users may not click in to see those instructions in the app.

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.

5 participants