Skip to content

Conversation

@hola-soy-milk
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Oct 10, 2024

Deploy Preview for directus-docs-v2 failed. Why did it fail? →

Name Link
🔨 Latest commit fc4d85c
🔍 Latest deploy log https://app.netlify.com/sites/directus-docs-v2/deploys/671220667aaff00008f0beb1

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. Mostly small changes. Global comments:

  • Watch random capitalization.
  • TODO all comments with images that need creating so we don't lose them.


<!-- TODO: Image -->

The following dynamic variables are built into Directus to add extra functionality to filters: `$CURRENT_USER`, `$CURRENT_ROLE`, `$CURRENT_ROLES`, `$CURRENT_POLICIES`, `$NOW` and `$NOW(<adjustment>)`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Go through and explain what each one is. I'd also be clear that CURRENT_ROLES and CURRENT_POLICIES are arrays


## Layouts

You can [select a layout](/content/layouts) for displaying your collection. Note that restrictions will apply depending on your collection's [data model](/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.

I'd briefly explain more what these are. I know that it's a whole page, but I don't think this helps people understand them

## Layout Presets
<!-- TODO: Image -->

By selecting more than one item in the explorer will allow you to click on :icon{name="material-symbols:filter-list} and edit several items' fields at once.
Copy link
Contributor

Choose a reason for hiding this comment

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

Mention that this is to the same value.


<!-- TODO: Image of explorer with bookmarks -->

Bookmarks are custom layouts that can be displayed for different views of your collections.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not quite. They specifically include more than layouts - also config, visible fields, sorting, filtering, search.


Promoting a version will turn it into the main version.

Open the version you want to promote and select the "Promote Version" option from the dropdown. In the "Changes" tab, you can review all the changes made in the version and decide which changes to accept or reject. Switch to the "Preview" tab to see a preview of the changes you are about to promote.
Copy link
Contributor

Choose a reason for hiding this comment

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

More here - for each field you can choose to include it in the promotion or leave it. I'd add an image here too. It's a cool interface.

Comment on lines 66 to 67
Some Directus users combat this by periodically purging some or all data in this collection. Be aware that this could
unintentionally purge content versions.
Copy link
Contributor

Choose a reason for hiding this comment

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

This feels scary. It's important, but is there any way we can soften this?

Copy link
Contributor

Choose a reason for hiding this comment

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

This page also needs, somewhere, mention that version is a dynamic variable that can be added to the live preview URL so you can preview a specific version. And then link back maybe to live preview?

Comment on lines 40 to 42
There are two ways to change the app language. First, administrators can set the project's
[default language](/configuration/translations). Second, users can choose their own personal
language preference.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove first and second. Conciseness is key.

hola-soy-milk and others added 11 commits October 16, 2024 12:18
Co-authored-by: Kevin Lewis <kvn@lws.io>
Co-authored-by: Kevin Lewis <kvn@lws.io>
Co-authored-by: Kevin Lewis <kvn@lws.io>
Co-authored-by: Kevin Lewis <kvn@lws.io>
Co-authored-by: Kevin Lewis <kvn@lws.io>
Co-authored-by: Kevin Lewis <kvn@lws.io>
Co-authored-by: Kevin Lewis <kvn@lws.io>
@hola-soy-milk
Copy link
Contributor Author

Thanks for the review! I think I've caught everything here. Can you please take another look?

Comment on lines 12 to 13
- **Version**: a version is a snapshot of the differences between the original item and the created version. Each
- version represents a set of future changes to be applied to the item.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is outstanding

@phazonoverload phazonoverload merged commit efcbfe7 into main Oct 18, 2024
@phazonoverload phazonoverload deleted the writeup/content branch October 18, 2024 08:46
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