Skip to content

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

Open
uttej-vsk wants to merge 16 commits into
BuilderIO:mainfrom
uttej-vsk:pr-3954
Open

Snippets: remove all references to getBuilderSearchParams ref to PR #3954#4078
uttej-vsk wants to merge 16 commits into
BuilderIO:mainfrom
uttej-vsk:pr-3954

Conversation

@uttej-vsk

Copy link
Copy Markdown
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.

@changeset-bot

changeset-bot Bot commented May 18, 2025

Copy link
Copy Markdown

🦋 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

@nx-cloud

nx-cloud Bot commented May 18, 2025

Copy link
Copy Markdown

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

@gitguardian

gitguardian Bot commented Jun 3, 2025

Copy link
Copy Markdown

️✅ 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.

Comment thread packages/sdks/snippets/angular-16/src/app/live-preview/live-preview.component.ts Outdated
Comment thread packages/sdks/snippets/qwik-city/src/routes/custom-child/index.tsx Outdated
@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
kapunahelewong added a commit that referenced this pull request Aug 20, 2025
## Description

This PR is a rebased and squashed update of
#4078 that I'm taking over. It
adds code snippets for the docs to the repo and was previously approved,
but the author hadn't signed their commits.

Co-authored-by: Uttej V S K <uttejvsk@hotmail.com>
murrayadam pushed a commit to murrayadam/builder that referenced this pull request Sep 5, 2025
## Description

This PR is a rebased and squashed update of
BuilderIO#4078 that I'm taking over. It
adds code snippets for the docs to the repo and was previously approved,
but the author hadn't signed their commits.

Co-authored-by: Uttej V S K <uttejvsk@hotmail.com>
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