Checks run (2 succeeded)
flake8
passed.
JSHint
passed.
Add the new "Connected Services" page.
Review Request #15139 — Created June 26, 2026 and updated
This adds the beginning of the new "Connected Service" page to the admin
site, replacing the old "Hosting Service Accounts" list. This currently
queries the hosting service accounts, groups them by service, and
renders a list of entries which link to the model form. The rendering
itself is designed to allow individual services to override the display
to add additional functionality.Next steps include:
- Make the "Connect a service" button do something.
- Link through to repository management, including listing out
repositories connected to eachHostingServiceAccountand a link to
create new repositories with the hosting service/account prepopulated.
- Enhance GitHub to be able to use the GitHub App manifest flow.
- Add a filtering UI for when there are many entries visible.
- Merge integration configuration into this page.
- Ran unit tests.
- Created some HostingServiceAccount entries for various services and
validated the result.
| Summary | ID |
|---|---|
| oquwuouprutzzynkvolvttxwqtxoworv |
| Description | From | Last Updated |
|---|---|---|
|
Eventually do we want this page to somehow make its way into RBCommons and replace the Repositories + Integrations Team … |
maubin
|
|
|
In the filenames, variable names and documentation we switch between "admin services", "connected services" or just "services". Can we standardize … |
maubin
|
|
|
Can you add "... repository_count attribute for accounts that support repositories ..." (even though its pretty self explanatory). |
maubin
|
|
|
Should we log something here if this attr isn't found. |
maubin
|
|
|
I think this is meant to look like <li class="rb-c-admin-cs-card">...</li> [0+] to denote that there can be 0 or more … |
maubin
|
|
|
This should be <div class="rb-c-admin-cs-card__body"> <ul class="rb-c-admin-cs-accounts"> <li class="rb-c-admin-cs-accounts__account">...</li> [0+] </ul> </div> |
maubin
|
|
|
This is meant to be <li class="rb-c-admin-cs-accounts__account">. |
maubin
|
|
|
This class is meant to be rb-c-admin-cs-accounts__account__detail. |
maubin
|
|
|
I think this doesn't exist anymore? |
maubin
|
-
-
Eventually do we want this page to somehow make its way into RBCommons and replace the Repositories + Integrations Team administration pages?
-
In the filenames, variable names and documentation we switch between "admin services", "connected services" or just "services". Can we standardize on "connected services", e.g.:
render_admin_services_list_entry->render_connected_services_list_entry
services_list_entry_template->connected_services_list_entry_template
and renaming similar items. -
Can you add "...
repository_countattribute for accounts that support repositories ..." (even though its pretty self explanatory). -
-
I think this is meant to look like
<li class="rb-c-admin-cs-card">...</li> [0+]to denote that there can be 0 or more
<li>entries. -
This should be
<div class="rb-c-admin-cs-card__body"> <ul class="rb-c-admin-cs-accounts"> <li class="rb-c-admin-cs-accounts__account">...</li> [0+] </ul> </div> -
-
-
Review request changed
- Commits:
-
Summary ID oquwuouprutzzynkvolvttxwqtxoworv oquwuouprutzzynkvolvttxwqtxoworv - Diff:
-
Revision 2 (+1652 -40)
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.