Skip to content

feat: Add updates in API v10 - WPB-18970 #3388

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

Open
wants to merge 6 commits into
base: feature/migrate-to-API-V10-WPB-18970
Choose a base branch
from

Conversation

dmitrysimkin
Copy link
Contributor

@dmitrysimkin dmitrysimkin commented Jul 24, 2025

Issue

Added changes for network requests that we use

https://wearezeta.atlassian.net/wiki/spaces/ENGINEERIN/pages/1984856066/API+changes+v9+v10

  • Added skip_creator to POST /conversations request
  • made POST /conversations (any some other with same model response) members.self optional
  • Conversation.id was remove in v10 but we already have it optional for other api versions with logic to recover id from qualifiedID
  • Updated POST /get-domain-registration with new response model

Testing

Describe how to test.

Optional: attachments like images, videos, etc.


Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

Copy link
Contributor

Test Results

    3 files    276 suites   1m 17s ⏱️
1 908 tests 1 907 ✅ 0 💤 1 ❌
1 909 runs  1 909 ✅ 0 💤 0 ❌

For more details on these failures, see this check.

Results for commit f0e9619.

@@ -21,4 +21,36 @@ import Foundation
final class AuthenticationAPIV10: AuthenticationAPIV9 {

override var apiVersion: APIVersion { .v10 }

override func getDomainRegistration(forEmail email: String) async throws -> DomainRegistrationConfiguration {
let path = "/get-domain-registration"
Copy link
Contributor

Choose a reason for hiding this comment

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

question: I guess we don't need pathPrefix, right?

@@ -21,4 +21,36 @@ import Foundation
final class AuthenticationAPIV10: AuthenticationAPIV9 {

override var apiVersion: APIVersion { .v10 }

override func getDomainRegistration(forEmail email: String) async throws -> DomainRegistrationConfiguration {
Copy link
Contributor

Choose a reason for hiding this comment

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

@netbe or @dmitrysimkin do you know the feature where we will use this new endpoint?

@KaterinaWire
Copy link
Contributor

@dmitrysimkin will you also add tests?

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