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

Prevent Duplicate Key Error when 'key' is passed into constructor. #3

Merged
merged 2 commits into from Apr 23, 2021
Merged

Prevent Duplicate Key Error when 'key' is passed into constructor. #3

merged 2 commits into from Apr 23, 2021

Conversation

tkeithblack
Copy link
Contributor

This change fixed an error with a duplicate key when the LinkWell widget is constructed passing in the 'key' value.

Error was:
Multiple widgets used the same GlobalKey.

This error never occurred unless the caller passed in a Key. It is easy in the old version to create this problem by just passing in a key in the constructor. The new version works correctly when passing in a key.

This change fixed an error with a duplicate key when the LinkWell widget is constructed passing in the 'key' value.
Error was:
Multiple widgets used the same GlobalKey.
Upgrading version number in podspec so Linkwell can be used with the url_launcher that supports null safety.

Note, this change will force apps to also update url_launcher & link_text to the null safety versions.
@tkeithblack
Copy link
Contributor Author

Also added a commit upgrading version number in podspec so Linkwell can be used with the url_launcher that supports null safety.

Note, this change will force apps to also update url_launcher & link_text to the null safety versions.

@samuelezedi samuelezedi merged commit 8c1a054 into samuelezedi:master Apr 23, 2021
@samuelezedi
Copy link
Owner

Great work man. Sorry I didn't see this until now.
Thank you for your contribution.

@tkeithblack
Copy link
Contributor Author

Cool, thanks for merging.

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.

None yet

2 participants