Skip to content

Snippets: remove all references to getBuilderSearchParams ref to PR #3954 #4078

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

uttej-vsk
Copy link
Contributor

Description

As per https://builder-io.atlassian.net/browse/ENG-8635?focusedCommentId=28672 we want to remove this helper from our snippets as it's unneeded.

Copy link

changeset-bot bot commented May 18, 2025

🦋 Changeset detected

Latest commit: 7819971

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

This PR includes changesets to release 8 packages
Name Type
@builder.io/sdk-angular Patch
@builder.io/sdk-react-nextjs Patch
@builder.io/sdk-qwik Patch
@builder.io/sdk-react Patch
@builder.io/sdk-react-native Patch
@builder.io/sdk-solid Patch
@builder.io/sdk-svelte Patch
@builder.io/sdk-vue Patch

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

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

nx-cloud bot commented May 18, 2025

View your CI Pipeline Execution ↗ for commit 7819971.

Command Status Duration Result
nx test @e2e/qwik-city ✅ Succeeded 9m 58s View ↗
nx test @e2e/nuxt ✅ Succeeded 8m 49s View ↗
nx test @e2e/nextjs-sdk-next-app ✅ Succeeded 8m 40s View ↗
nx test @e2e/angular-17 ✅ Succeeded 7m 54s View ↗
nx test @e2e/angular-17-ssr ✅ Succeeded 6m 39s View ↗
nx test @e2e/react-sdk-next-15-app ✅ Succeeded 6m 34s View ↗
nx test @e2e/react-sdk-next-14-app ✅ Succeeded 6m 12s View ↗
nx test @e2e/angular-19-ssr ✅ Succeeded 6m 33s View ↗
Additional runs (37) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2025-06-20 20:08:43 UTC

Copy link

gitguardian bot commented Jun 3, 2025

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@uttej-vsk uttej-vsk marked this pull request as ready for review June 10, 2025 17:34
@uttej-vsk uttej-vsk requested review from clyde-builderio and removed request for builderio-bot June 10, 2025 17:36
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