Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

We have a number of import errors occurring with 422 errors, and
it's hard to determine why they are happening. This change will
surface the errors in the log lines.

Relates to #47365

There's no need to add a redundant message to the errors if the
model is invalid. This cleans up the message as well for the importer.

Importing a project from GitHub for a project namespace that already exists
would show an unhelpful error, "An error occurred while importing project."
We now add the base message from Projects::CreateService when this fails.

Closes #47365

#close_merge_request_without_source_project -> close_merge_request_if_no_source_project

This allows us to report JSON parse exceptions to clients and ignore
them in sentry.

When we want to show an access denied message to a user, we don't have
to hide the resource's existence.

So in that case we render a 403, this 403 is not handled by nginx on
omnibus installs, making sure the message is visible to the user.

We show a blue flash banner if the user already accepted, and show a
button allowing them to continue to the application.

Since we don't show the archived projects, we shouldnot  load them and
pass them to the fronted to be filtered out again.

"Maintainer" will be freed to be used for #42751

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

This endpoint lists projects shared with a group visible to the
current user.

The `filter` and `sort` params are supported like on the `children` endpoint.

This commit does a number of things:

1. Reduces the number of queries needed by perform a single query to get all
the tuples for the relevant rows.

2. Uses a transaction to query the tuple counts to ensure that the data
is retrieved from the primary.

Closes #46742

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

this required moving some data store actions & mutations around

Signed-off-by: NRémy Coutable <remy@rymai.me>