Skip to content

feat: formbricks v5 changes#61

Merged
pandeymangg merged 3 commits into
mainfrom
epic/v5
May 18, 2026
Merged

feat: formbricks v5 changes#61
pandeymangg merged 3 commits into
mainfrom
epic/v5

Conversation

@pandeymangg
Copy link
Copy Markdown
Contributor

Merging the epic/v5 changes for formbricks v5 into main

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 18, 2026

CLA assistant check
All committers have signed the CLA.

@pandeymangg pandeymangg requested a review from Dhruwang May 18, 2026 06:00
@sonarqubecloud
Copy link
Copy Markdown

@pandeymangg pandeymangg added this pull request to the merge queue May 18, 2026
Merged via the queue into main with commit 80a313f May 18, 2026
9 of 10 checks passed
@pandeymangg pandeymangg deleted the epic/v5 branch May 18, 2026 06:06
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 18, 2026

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

This pull request migrates the React Native SDK from an environmentId-based architecture to a workspaceId-first model. The migration introduces workspace-scoped state management, updates API contracts to workspace endpoints, adds configuration migration for backward compatibility, and refactors all components and utilities to operate on workspace state instead of environment state. The changes span type definitions, the API client, a new workspace state module with expiry listeners, setup orchestration logic, event listener wiring, survey filtering and styling utilities, component entry points, user update flows, and comprehensive test coverage validating the new workspace-centric behavior.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 60.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'feat: formbricks v5 changes' is vague and generic, using non-descriptive terms that don't convey meaningful information about the specific changeset. Use a more descriptive title that highlights the primary architectural change, such as 'feat: migrate from environment-based to workspace-based API' or 'feat: replace environmentId with workspaceId throughout React Native SDK'.
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description relates to the changeset by indicating this is a major version upgrade (v5) that refactors the SDK's core configuration model from environment-centric to workspace-centric.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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