Skip to content

Conversation

@hola-soy-milk
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Oct 7, 2024

Deploy Preview for directus-docs-v2 ready!

Name Link
🔨 Latest commit b61c355
🔍 Latest deploy log https://app.netlify.com/sites/directus-docs-v2/deploys/6706645e61d0750008fd2ee7
😎 Deploy Preview https://deploy-preview-36--directus-docs-v2.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.

@hola-soy-milk hola-soy-milk marked this pull request as ready for review October 7, 2024 12:15
Copy link
Contributor

@phazonoverload phazonoverload left a comment

Choose a reason for hiding this comment

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

Good. Question I'm not sure about - isn't this also a setting inside of flow configuration - whether to log activities or not?


<!-- TODO: Image -->

The activity log module provides a collective timeline of all data-changing actions taken within your project. It is accessed via the notifications tray of the sidebar. The page shares the filtering and search with the [Collection Page](/data-modeling/collections).
Copy link
Contributor

Choose a reason for hiding this comment

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

The page shares the filtering and search with the Collection Page

Really struggled to read this - took me a few times. Worth revision?

Comment on lines 11 to 18
::callout{type="warning" url="/auth/creating-users" title="External Changes"}

Read more about creating users.

Directus can only track the events which actually pass through its platform. Therefore, any changes made to the database
directly are not tracked.

::
Copy link
Contributor

Choose a reason for hiding this comment

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

What's going on here? I think line 13 needs removing?


- **User** — The user that performed the action.
- **Action** — The specific action taken (e.g., Create, Update, Delete, Comment, or Login).
- **TimeStamp** — The timestamp of when the action was performed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure TimeStamp is capitalized like this

@hola-soy-milk
Copy link
Contributor Author

Thanks for the feedback @phazonoverload! Can you please take another look?

Comment on lines 11 to 17
::callout{type="warning" url="/auth/creating-users" title="External Changes"}

Read more about creating users.

Directus can only track the events which actually pass through its platform. Therefore, any changes directly made to the database are not tracked in the activity log.

::
Copy link
Contributor

Choose a reason for hiding this comment

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

Confused about this still

  • URL - creating users
  • title - external changes
  • content - bit of both

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for pointing this out I don't know what happened here.

@hola-soy-milk
Copy link
Contributor Author

Thank you for pointing out this error on my end! Should be good to go I hope

@phazonoverload phazonoverload merged commit 9865755 into main Oct 9, 2024
5 of 6 checks passed
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