Skip to content

fix: only import workflows for new users + add Explore#63

Merged
acedatacloud-dev merged 1 commit intomainfrom
fix/workflow-import-and-explore
Mar 14, 2026
Merged

fix: only import workflows for new users + add Explore#63
acedatacloud-dev merged 1 commit intomainfrom
fix/workflow-import-and-explore

Conversation

@acedatacloud-dev
Copy link

Changes

  1. New users: Import all 13 workflow templates into their workspace on first login
  2. Existing users: Skip workspace import — preserves existing apps untouched
  3. Explore section: Register workflow templates as RecommendedApps so they appear in Explore for all users

How it works

  • oauth_new_user flag passed from OAuth callback to import task
  • Explore setup is idempotent (Redis guard key)
  • First login triggers Explore setup; subsequent logins skip it
  • Workflows registered as RecommendedApp with is_public=True

- Pass is_new_user flag from OAuth to import task
- New users: import all workflow templates into workspace
- Existing users: skip workspace import (preserve existing apps)
- Register workflow templates as RecommendedApps in Explore
- Idempotent Explore setup guarded by Redis key
@github-actions
Copy link

Pyrefly Diff

base → PR
3256,3260c3256,3260
<    --> tasks/import_acedatacloud_workflow_templates_task.py:135:17
<     |
< 135 |                 account.current_tenant_id = tenant_id
<     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^
<     |
---
>   --> tasks/import_acedatacloud_workflow_templates_task.py:86:5
>    |
> 86 |     account.current_tenant_id = tenant_id
>    |     ^^^^^^^^^^^^^^^^^^^^^^^^^
>    |

1 similar comment
@github-actions
Copy link

Pyrefly Diff

base → PR
3256,3260c3256,3260
<    --> tasks/import_acedatacloud_workflow_templates_task.py:135:17
<     |
< 135 |                 account.current_tenant_id = tenant_id
<     |                 ^^^^^^^^^^^^^^^^^^^^^^^^^
<     |
---
>   --> tasks/import_acedatacloud_workflow_templates_task.py:86:5
>    |
> 86 |     account.current_tenant_id = tenant_id
>    |     ^^^^^^^^^^^^^^^^^^^^^^^^^
>    |

Copy link
Member

@Germey Germey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@acedatacloud-dev acedatacloud-dev merged commit 1c9337d into main Mar 14, 2026
17 of 18 checks passed
@acedatacloud-dev acedatacloud-dev deleted the fix/workflow-import-and-explore branch March 14, 2026 13:34
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