chore: sync upstream PR #8629 - fix(http): keep the existing query string when adding params on web - #167
chore: sync upstream PR #8629 - fix(http): keep the existing query string when adding params on web#167riderx wants to merge 39 commits into
Conversation
Co-authored-by: jcesarmobile <jcesarmobile@gmail.com>
Co-authored-by: Joey Pender <joey.pender@outsystems.com> Co-authored-by: Pedro Bilro <pedro.gustavo.bilro@outsystems.com> Co-authored-by: Mark Anderson <mark.anderson@outsystems.com>
…#8476) Co-authored-by: jcesarmobile <jcesarmobile@gmail.com>
…am#8492) Co-authored-by: Mark Anderson <mark.anderson@outsystems.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…-team#8271) Co-authored-by: Eric Horodyski <horodyski@ionic.io>
…ermissions (ionic-team#8400) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Pedro Bilro <pedro.gustavo.bilro@outsystems.com>
The web CapacitorHttp request joined params with '?' even when the url already had a query string, producing url?a=1?b=2. Use '&' in that case, matching Android and iOS. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ferred conflicts)
|
Git applied the upstream-preferred strategy to resolve this sync. Please review the branch carefully before merging. |
Beta npm buildMaintainers can publish one Capacitor Plus workspace package from this PR to npm for fast testing. Comment Examples: /publish-beta core
/publish-beta cli
/publish-beta @capacitor-plus/coreIf exactly one workspace package changed, Packages:
The workflow will:
Security note: beta publish is only enabled for branches inside this repository. |
|
Warning Review limit reachedNext included review available in 4 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (47)
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. Comment |
Merge Conflict Review Required
The sync of upstream PR ionic-team#8629 from @breken-ai encountered merge conflicts.
Original PR: ionic-team#8629
What happened
Synced from upstream by Capacitor+ Bot
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.