kobotoolbox / kpi / 25077598537

82%

master: 76%

push

github

fix(connectProjects): deleted imported project issue DEV-2026 (#6971)

### 📣 Summary

Fixes a problem with Connect Projects UI when an imported project was
removed before the import-link was deleted. The problem was causing an
infinite loading spinner (or crash).

Also migrated few old icons and buttons to latest components. Also
introduced a confirmation for imported project removing.

### 💭 Notes

I think the issue reported on Linear was a bit different from issue on
`main` branch given [the recent connectProjects
PRs](https://github.com/kobotoolbox/kpi/pulls?q=is%3Aclosed%20is%3Apr%20author%3Amagicznyleszek%20connectProjects).
When I tested it, I was getting a crash, and Linear issue describes
infinite spinner (in the same area though).

I allowed myself to polish the UI a bit, as it was bugging me while
debugging ;)

Changes here:
- Fixed the issue → mainly problem was that after removing project
`source.source__name` is `null`, and code tried calling
`truncateString()` on it expecting a string
- Migrated `<i>` to `<Icon>` component
- For deleted imported project, I show different icon to make it clearer
- Deleted imported project has now disabled "Configure" button and a
description override
- Migrated `<Button>` to `<ActionIcon>` and `<ButtonNew>`
- When removing an imported project, we now display confirmation modal

I recommend "Hide whitespace" in "Files changed" tab settings

### 👀 Preview steps

See steps [described at Linear
task](https://linear.app/kobotoolbox/issue/DEV-2026/deleting-a-connected-project-causes-infinite-load-when-trying-to#steps-to-reproduce-bd538fc9)

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: James Kiger <68701146+jamesrkiger@users.noreply.github.com>

9048 of 12138 branches covered (74.54%)

29832 of 36300 relevant lines covered (82.18%)

5.78 hits per line