Admin UI: minor cleanup to useList

[Vultraz]

• Removed unnecessary manual query refetches (already handled via refetchQueries)
• Removed the query property from useList
• Added loading and error to useList
• Stop passing query through to ListLayout via prop

[changeset-bot]

🦋 Changeset is good to go

Latest commit: 0b56293

We got this.

This PR includes changesets to release 1 package

Name	Type
@keystonejs/app-admin-ui	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

[gautamsi]

Hey, query is needed for re-fetching manually after making changes in listmanageActions UI hooks, I am sure you can suggest alternative. should this be refetchQueries in my mutations?

[Vultraz]

refetchQueries is the pattern I've been using, but I can add refetch directly to the hook if you would prefer to use that.

[gautamsi]

Let me try that, we would have to document this.

[gautamsi]

I think I am fine with the refetchQueries approach, go ahead if you were waiting for my feedback.

[stale]

It looks like there hasn't been any activity here in over 6 months. Sorry about that! We've flagged this issue for special attention. It wil be manually reviewed by maintainers, not automatically closed. If you have any additional information please leave us a comment. It really helps! Thank you for you contribution. :)

[bladey]

Thanks for this PR @Vultraz.

Keystone 5 has officially moved into active maintenance mode as we push towards the next major new version Keystone Next, you can find out more information about this transition here.

In an effort to sustain the project going forward, we're cleaning up and closing old PRs such as this one.

However, you are welcome to create a PR under the Keystone 5 repository.

Please let me know if you have any questions.