Skip to content

Fix tencent translate#1008

Merged
tisfeng merged 4 commits intodevfrom
fix/tencent-translate
Oct 19, 2025
Merged

Fix tencent translate#1008
tisfeng merged 4 commits intodevfrom
fix/tencent-translate

Conversation

@tisfeng
Copy link
Copy Markdown
Owner

@tisfeng tisfeng commented Oct 6, 2025

Try to fix #975

Updates the Alamofire dependency to the latest version, 5.10.2.

This also adjusts the package requirement in the Xcode project from a fixed `exactVersion` to `upToNextMajorVersion` to allow for easier minor and patch version updates in the future.
Replaces the explicit dictionary type `[String: Any]` with Alamofire's `Parameters` typealias in the Tencent and Volcano translation services.

This is a minor code quality improvement that enhances readability and maintains consistency with the Alamofire networking library's conventions.
@tisfeng tisfeng marked this pull request as draft October 6, 2025 18:49
Corrects an inconsistency by replacing `[String: Any]` with the `Parameters` typealias for improved code consistency.

This commit updates the function signatures within the `TencentSigning` and `VolcanoSigning` logic to match the changes made in the service layer, ensuring the `Parameters` type is used throughout the request and signing process.
…ao services

- Change type declaration from [String: Any] to Parameters in CaiyunService.swift
- Update generalParameters type from [String: Any] to Parameters in YoudaoService+Translate.swift
- Ensures consistency with Alamofire's parameters parameter type
@tisfeng
Copy link
Copy Markdown
Owner Author

tisfeng commented Oct 19, 2025

Referring to the reply in Alamofire/Alamofire#3983 , I fixed the fundamental issue that upgrading Alamofire would cause the Tencent translation to fail.

Everyone please note that when we use Alamofire later, parameters should use the Parameters type, not [String: Any].

@tisfeng tisfeng merged commit 7a9386d into dev Oct 19, 2025
4 checks passed
@tisfeng tisfeng deleted the fix/tencent-translate branch October 19, 2025 13:54
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.

更新到最新版本后腾讯翻译使用不了了!

2 participants