Skip to content

Cleanup process builder header#600

Open
nourmalaeb wants to merge 4 commits intodevfrom
cleanup-process-builder-header
Open

Cleanup process builder header#600
nourmalaeb wants to merge 4 commits intodevfrom
cleanup-process-builder-header

Conversation

@nourmalaeb
Copy link
Collaborator

@nourmalaeb nourmalaeb commented Feb 13, 2026

  • Render a clean header (no nav or validation or launch button) when selecting a process template
  • Display correct name in the header when editing a process
  • Display correct name in modal when launching a process

@nourmalaeb nourmalaeb changed the base branch from dev to wire-up-launch-process-button February 13, 2026 06:16
@vercel
Copy link
Contributor

vercel bot commented Feb 13, 2026

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

Project Deployment Actions Updated (UTC)
app Ready Ready Preview Feb 13, 2026 10:01pm

Request Review

Copy link
Collaborator

@scazan scazan left a comment

Choose a reason for hiding this comment

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

Approving but let's make sure we are pulling the right name from the profile before merge.

const storeInstanceName = useProcessBuilderStore((state) =>
decisionProfileId ? state.instances[decisionProfileId]?.name : undefined,
);
const displayName = instance?.name || storeInstanceName || t('New process');
Copy link
Collaborator

Choose a reason for hiding this comment

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

we should get this off the profile rather than the instance name. it is unfortunately a side effect of legacy data that this field exists. I will submit a removal PR for the column.

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