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.

Sign up for GitHub

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

Jump to bottom

migrate from ui.hash* to Object.hash* #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

migrate from ui.hash* to Object.hash* #58

wants to merge 2 commits into from

Conversation

zeqinjie
Copy link

@zeqinjie zeqinjie commented Jul 20, 2024 •
edited
Loading

  • hashValues was deprecated in v3.1.0-0.0.pre.897'
image image

Pre-launch Checklist

  • I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
  • I updated CHANGELOG.md to add a description of the change.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making if a test is possible.
  • All existing and new tests are passing.

@zeqinjie zeqinjie force-pushed the fix_hashValues_deprecate branch from 5d14d3d to 75b89c1 Compare July 20, 2024 06:48
@Kiruel
Copy link

Kiruel commented Dec 12, 2024

Can we merge this one, we got this problem on the new version of flutter stable 3.27.0.

troya2
troya2 approved these changes Dec 13, 2024
View reviewed changes
Copy link

@troya2 troya2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code is clear and single purposed - change hashValues to Object.hash and update the version of the project.

@zeqinjie
Copy link
Author

@LuisThein Can you review the code?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@troya2 troya2 troya2 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zeqinjie @Kiruel @troya2