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(useCookieState): batch updates #2084

Merged
merged 2 commits into from
Mar 1, 2023
Merged

Conversation

liuyib
Copy link
Collaborator

@liuyib liuyib commented Mar 1, 2023

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Internationalization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

同:#2082, 如果 2082 这个 PR 可以被合并,那么 useCookieState 也需要做对应的改动,使得它们的行为保持一致

📝 Changelog

Language Changelog
🇺🇸 English fix batch updates of useCookieState
🇨🇳 Chinese 修复 useCookieState 批量更新时的行为

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

@liuyib
Copy link
Collaborator Author

liuyib commented Mar 1, 2023

此 hook 改动前后的表现行为,同 2082,见:#2082 (comment)

@liuyib liuyib requested a review from crazylxr March 1, 2023 02:03
@liuyib liuyib added the v4 label Mar 1, 2023
@crazylxr crazylxr merged commit 1f6ac82 into master Mar 1, 2023
@crazylxr crazylxr deleted the fix_useCookieState_batch_update branch March 1, 2023 11:21
crazylxr added a commit that referenced this pull request Mar 1, 2023
crazylxr added a commit that referenced this pull request Mar 1, 2023
@liuyib liuyib restored the fix_useCookieState_batch_update branch May 10, 2024 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants