Skip to content

fix(iOS): change hardcoded config path when using custom config file for auto register plugins#7540

Open
tossaro wants to merge 2 commits intoionic-team:mainfrom
tossaro:bugfix/ios-auto-register-plugin-with-custom-config-file
Open

fix(iOS): change hardcoded config path when using custom config file for auto register plugins#7540
tossaro wants to merge 2 commits intoionic-team:mainfrom
tossaro:bugfix/ios-auto-register-plugin-with-custom-config-file

Conversation

@tossaro
Copy link

@tossaro tossaro commented Jun 30, 2024

When I use InstanceDescriptor with custom config path, the CapacitorBridge initialization on the CAPBridgeViewController is by default autoRegisterPlugins set to true.
But as we can see on CapacitorBridge class function registerPlugins config path is hardcoded
if autoRegisterPlugins { do { if let pluginJSON = Bundle.main.url(forResource: "capacitor.config", withExtension: "json") { //..
So, I've created new property configUrl that will be given when parsing configuration at InstanceDescriptor initialization.

@markemer markemer self-requested a review February 25, 2025 19:28
@markemer markemer self-assigned this Feb 25, 2025
@eric-horodyski
Copy link
Member

Thanks for opening this PR 🙏

As part of our backlog maintenance effort, we’re reviewing older pull requests for compatibility with the current branch structure and latest releases.

Details: https://ionic.io/blog/keeping-capacitors-backlog-healthy

This PR will need to be rebased against the latest main branch and updated to resolve merge conflicts.

If we don’t see updates within 7 days, we’ll close it to keep the queue focused — but it can always be resubmitted once updated.

@tossaro tossaro force-pushed the bugfix/ios-auto-register-plugin-with-custom-config-file branch from 5d1906d to 5f36929 Compare February 26, 2026 17:39
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.

4 participants