Skip to content

feat(apollo-react): loading state for toolbox#276

Merged
uipath-utkarsh merged 1 commit intomainfrom
MST-7042
Feb 27, 2026
Merged

feat(apollo-react): loading state for toolbox#276
uipath-utkarsh merged 1 commit intomainfrom
MST-7042

Conversation

@uipath-utkarsh
Copy link
Contributor

Description

Use loading state skeleton when + is clicked on stage node if the APIs are still in flight.

Once the APIs are successful, populate the toolbox.

  1. Repurposed addTaskLoading prop on StageNode to drive loading state inside the Toolbox instead of showing a spinner on the + button. This is a breaking change
  2. The + button is now always clickable and shows the add icon regardless of addTaskLoading — loading is delegated to the Toolbox's skeleton/disabled-items UI.
  3. Updated the AddTaskLoading story to demonstrate both loading patterns:
  4. Top-level loading: addTaskLoading=true + empty taskOptions → Toolbox shows skeleton loaders until API returns items
  5. Async children loading: ListItem with async children function → Toolbox internally dims level 1 items while fetching level 2 subtasks (no extra props needed)
  6. Removed unused ApCircularProgress import from StageNode.tsx
  7. Updated tests to verify the new behavior (loading passed to Toolbox, button no longer disabled)

Demo

Screen.Recording.2026-02-27.at.4.53.45.PM.mov

@github-actions
Copy link

github-actions bot commented Feb 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (PT)
apollo-canvas 🟢 Ready Preview, Logs Feb 27, 2026, 03:30:17 AM
apollo-ui-react 🟢 Ready Preview, Logs Feb 27, 2026, 03:29:00 AM
apollo-vertex 🟢 Ready Preview, Logs Feb 27, 2026, 03:28:35 AM
apollo-wind 🟢 Ready Preview, Logs Feb 27, 2026, 03:27:27 AM

@github-actions
Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@ayushya
Copy link
Contributor

ayushya commented Feb 27, 2026

Primarily, this PR fixes what we were trying to do.

As a follow-up on this PR, if you also achieve the following, it would save us from creating another ticket later.
In the demo, when you click the Email item, it should immediately jump to email and show a loader within its items section until the data is loaded

@uipath-utkarsh uipath-utkarsh merged commit f140071 into main Feb 27, 2026
24 checks passed
@uipath-utkarsh uipath-utkarsh deleted the MST-7042 branch February 27, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants