Skip to content
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

fix(connectivity_plus): Return valid connection type when only one available #2668

Merged
merged 1 commit into from
Mar 10, 2024

Conversation

vbuberen
Copy link
Collaborator

Description

I was working on updates for example app to show newly added multiple connection types feature in a more nice way and found a bug that when only one connection type is available there is an exception in the platform interface as returned result isn't a list, but a String:
Screenshot 2024-03-10 at 13 48 22

This PR fixes mentioned bug and also updates the example UI to look better:

Before

photo_2024-03-10 14 18 31

After

photo_2024-03-10 14 18 29

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

  • Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
  • No, this is not a breaking change.

@miquelbeltran
Copy link
Member

On which platform did you get this error? I would like to look it up and ensure that the stream always returns an array

@vbuberen
Copy link
Collaborator Author

On Android

@vbuberen vbuberen merged commit 81026a4 into main Mar 10, 2024
19 checks passed
@vbuberen vbuberen deleted the update/connectivity_example branch March 10, 2024 21:35
@asonni
Copy link

asonni commented Mar 24, 2024

I hope this issue will be fixed soon.

@vbuberen
Copy link
Collaborator Author

I hope this issue will be fixed soon.

Do you experience this issue? It never made it to the stable release of the plugin.

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