search

ynput / ayon-backend

Server codebase with API access to AYON
Apache License 2.0
22 stars 15 forks source link

Activity Feed: comment auto suggestions #224

Closed martastain closed 4 months ago

martastain commented 4 months ago

Implements server-side resolving for entity autosuggestion in comments

Innders commented 4 months ago

Is it too crazy to accept entityIds (plural)? You can select more than one entity of the same type at one time.

martastain commented 4 months ago

it is crazy... you are always commenting on one entity.

Innders commented 4 months ago

you are always commenting on one entity

Actually you can comment on multiple 😬 Okay let's leave it for now...

martastain commented 4 months ago

Users seems to be broken

Screen.Recording.2024-06-04.at.16.34.30.mov

Are those users assigned to anything?

Innders commented 4 months ago

Are those users assigned to anything?

Looks like Demo User is the only user assigned to other tasks in the project. All the other users are not assigned to anything.

If I add hotel user to any task on the project it will now appear in the search results.

I also think me should appear as well, I might want to tag myself.

Innders commented 4 months ago

Status could influence relevance as well.

Status states ranked by most relevant:

  1. In Progress
  2. Blocked
  3. Not started
  4. Done
martastain commented 4 months ago

Status could influence relevance as well.

Status states ranked by most relevant:

  1. In Progress
  2. Blocked
  3. Not started
  4. Done

I cannot use statuses in the query logic as they are configurable and may differ :(

Innders commented 4 months ago

I cannot use statuses in the query logic as they are configurable and may differ :(

But status states do not change?

image
martastain commented 4 months ago

I cannot use statuses in the query logic as they are configurable and may differ :(

But status states do not change? <img alt="image" width="1113" src="https://private-user-

That's true. Okay - it's doable, but I'm not convinced if it's worth another two subqueries. Let's keep that open as a "future extension" :)

BigRoy commented 4 months ago

Looks like Demo User is the only user assigned to other tasks in the project. All the other users are not assigned to anything.

If a user can access the project then I'd want to also be able to tag them. They may not be prioritized in the list but I'd be confused being incapable of tagging someone who's not explicitly assigned a task. E.g. our producers usually are not assigned a production task - but I do want to tag them if I want to notify them about something.

Innders commented 4 months ago

That's true. Okay - it's doable, but I'm not convinced if it's worth another two subqueries. Let's keep that open as a "future extension" :)

Yeah cool, just throwing ideas around 👍