Skip to content

Fix/rename flowId to flowExecId#443

Open
NutharaNR wants to merge 4 commits intoasgardeo:mainfrom
NutharaNR:fix/rename-flowId-to-flowExecId
Open

Fix/rename flowId to flowExecId#443
NutharaNR wants to merge 4 commits intoasgardeo:mainfrom
NutharaNR:fix/rename-flowId-to-flowExecId

Conversation

@NutharaNR
Copy link
Copy Markdown

@NutharaNR NutharaNR commented Apr 8, 2026

Purpose

Rename flowId in flow execution in thunder to flowExecId.

Related Issues

Related PRs

Checklist

  • Followed the CONTRIBUTING guidelines.
  • Manual test round performed and verified.
  • Documentation provided. (Add links if there are any)
  • Unit tests provided. (Add links if there are any)

Security checks

@NutharaNR NutharaNR force-pushed the fix/rename-flowId-to-flowExecId branch from 9c26c20 to 19dd39f Compare April 9, 2026 03:23
@NutharaNR NutharaNR marked this pull request as ready for review April 9, 2026 03:48
@NutharaNR NutharaNR force-pushed the fix/rename-flowId-to-flowExecId branch 4 times, most recently from 2aa8f21 to 19dd39f Compare April 9, 2026 06:44
@NutharaNR NutharaNR force-pushed the fix/rename-flowId-to-flowExecId branch from b5f75b0 to 2eb2da1 Compare April 9, 2026 07:17
@asgardeo-github-bot
Copy link
Copy Markdown

🦋 Changeset detected

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

@@ -0,0 +1,7 @@
---
'@asgardeo/javascript': patch
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think these has to be a minor release.
cc: @brionmario

currentFlowId,
flowIdStorageKey: FLOW_ID_STORAGE_KEY,
currentExecutionId,
executionIdStorageKey: FLOW_ID_STORAGE_KEY,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Will it makes sense to change FLOW_ID_STORAGE_KEY as well (both key and value)?
@brionmario WDYT?

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.

3 participants