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: i18n API no data #57

Merged
merged 1 commit into from
Dec 17, 2024
Merged

fix: i18n API no data #57

merged 1 commit into from
Dec 17, 2024

Conversation

LittleLollipop
Copy link
Contributor

In version 0.3.0, the introduction of the interface caching mechanism led to updates in several API implementations. However, the behavior regarding the names parameter being empty was not aligned with the previous method, resulting in data loss in the i18n interface. This commit addresses and fixes this issue.

Key Points:

  • Version Update: The changes were made in version 0.3.0.
  • Caching Mechanism: The new caching mechanism introduced modifications to API implementations.
  • Parameter Handling: The handling of the names parameter when empty was inconsistent with the old method.
  • Data Loss: This inconsistency caused data loss in the i18n interface.
  • Fix: The current commit resolves this issue by aligning the new method's behavior with the old one.

This update ensures that the i18n interface functions correctly, preventing any further data loss due to parameter handling discrepancies.

Copy link

vercel bot commented Dec 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
market-olares ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 17, 2024 10:01am
market-terminus ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 17, 2024 10:01am

@LittleLollipop LittleLollipop changed the title try fix i18n no data fix i18n API no data Dec 17, 2024
@LittleLollipop LittleLollipop changed the title fix i18n API no data fix: i18n API no data Dec 17, 2024
@github-actions github-actions bot added the fix label Dec 17, 2024
@icebergtsn icebergtsn merged commit 2ef10e6 into main Dec 17, 2024
4 of 6 checks passed
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.

2 participants