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

Pro HTTP client updates #1033

Merged
merged 95 commits into from
Apr 18, 2024
Merged

Pro HTTP client updates #1033

merged 95 commits into from
Apr 18, 2024

Conversation

atavism
Copy link
Contributor

@atavism atavism commented Apr 1, 2024

Depends on getlantern/flashlight#1371

@atavism
Copy link
Contributor Author

atavism commented Apr 18, 2024

@atavism have you completed your testing?

@jigar-f I just finished testing desktop and fixed a couple issues (such as the formatted monthly cost missing from plans for instance). Now the full checkout flow is working for me again (plans -> payment methods -> payment redirect).

I'll test Android in the morning.

@jigar-f
Copy link
Contributor

jigar-f commented Apr 18, 2024

@atavism Just a thought, Do you think we decouple proclient? So later on we can use that for Android and IOS as well, At the moment it looks like it is tightly coupled with desktop code, Maybe can create some custom interface and pass that NewClient so we can use it for desktop and android and IOS as well.

internalsdk/pro/pro.go Outdated Show resolved Hide resolved
Copy link
Contributor

@jigar-f jigar-f left a comment

Choose a reason for hiding this comment

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

LGTM, Just some comments.

@jigar-f
Copy link
Contributor

jigar-f commented Apr 18, 2024

@atavism

  • I do not see a loading bar when doing the Reseller checkout
  • When I click authorize device with Link with PIN It doesn't show the PIN

Screenshot 2024-04-18 at 5 54 05 PM

@atavism
Copy link
Contributor Author

atavism commented Apr 18, 2024

@atavism Just a thought, Do you think we decouple proclient? So later on we can use that for Android and IOS as well, At the moment it looks like it is tightly coupled with desktop code, Maybe can create some custom interface and pass that NewClient so we can use it for desktop and android and IOS as well.

That definitely makes sense. Making some changes now to no longer pass settings to ProClient

@atavism
Copy link
Contributor Author

atavism commented Apr 18, 2024

Maybe can create some custom interface and pass that NewClient so we can use it for desktop and android and IOS as well.

This is done. Updating iOS to use it now.

@atavism
Copy link
Contributor Author

atavism commented Apr 18, 2024

I do not see a loading bar when doing the Reseller checkout
When I click authorize device with Link with PIN It doesn't show the PIN

The device name was missing requesting a linking code:

{"status":"error","error":"Bad input: request missing Device Name","errorId":"bad-input-missing-device-name","Code":"","ExpireAt":0}

Should be fixed now.

@atavism
Copy link
Contributor Author

atavism commented Apr 18, 2024

Maybe can create some custom interface and pass that NewClient so we can use it for desktop and android and IOS as well.

iOS is using the new Pro client now.

For Android, the changes are more extensive than I anticipated since we have to get rid of the LanternHttpClient. I opened a ticket for that.

@atavism atavism merged commit 9713030 into main Apr 18, 2024
11 of 12 checks passed
@atavism atavism deleted the atavism/pro-client-updates branch April 18, 2024 22:47
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