directus icon indicating copy to clipboard operation
directus copied to clipboard

Allow Kanban titles to use referenced data

Open matt-rolley opened this issue 1 year ago • 2 comments

Scope

What's changed:

  • Ability to use a referenced collections display value in the Kanban title

Potential Risks / Drawbacks

  • Could potentially add/load more related fields then required?

Review Notes / Questions

  • I've not done much reference field loading in the directus app before, so if there is a better way to do it please let me know :)
  • Any review/advice would be greatly appreciated.

Fixes #23537

matt-rolley avatar Sep 02 '24 12:09 matt-rolley

🦋 Changeset detected

Latest commit: 7633443cd270ae7b6a48a12d478b958b1eddcf64

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@directus/app Minor
@directus/api Patch
directus Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Sep 02 '24 12:09 changeset-bot[bot]

Hey @matt-rolley! Thank you for the PR!

I made a couple of changes, that include fixing a regression, that would not allow normal text fields to be selected anymore and simplifying the addition of the fields necessary for rendering the display template.

hanneskuettner avatar Sep 23 '24 08:09 hanneskuettner

  • Fields that have relations only show up if they have a display template set, they should be selectable regardless if they have a display configured with a display template.

❗ I could not reproduce this. Is it still valid or can you provide more detailed steps to reproduce it?

  • Is this supposed to add support for only M2O or also O2M, M2M, and M2A? Only M2O works for me.

✅ Done

  • The styling needs some work, setting the title to be user_created for users who do not have a profile picture results in the default material icon being huge.

✅ Done

  • When I initially switched from the table to kanban layout, the cards did not load and instead were replaced by a red error message saying it couldn't read textField of undefined. Deselecting the title field and reselecting the same field made the error go away and the cards to load. This error only showed up on the initial time the kanban was loaded so there may need to be some logic to update existing configs? Not sure about this one.

❗ I could not reproduce this. Is it still valid or can you provide more detailed steps to reproduce it?


From a functional point of view, it looks good to me, at the same time I would strongly recommend a design review of the card's appearance by the creative team, as it seems to have changed a lot over time.

formfcw avatar Dec 12 '24 09:12 formfcw