Clean up callback flow for sign-in and verify flow.
@brnnmrls đang làm issue này rồi.
Từ ngày 21/5/2024.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Issue found in: https://github.com/google/GoogleSignIn-iOS/pull/414#discussion_r1607303702
The current sign-in and verify flow are brittle when processing the authorization response. The current flow will depend on an [authFlow wait] to ensure a token request is made before moving forward. Doing so makes the code harder to understand and easy to break. A way to improve this is by adding completion callbacks to ensure a tokenRequest is made before returning an instance of GIDAuthFlow.
- Ngôn ngữ chính
- Objective-C
- Star
- 751
- Fork
- 282
- Merge trung bình
- 2 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 9
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của google/GoogleSignIn-iOS
-
enhancement triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
google/GoogleSignIn-iOS#621 ·
-
enhancement triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
google/GoogleSignIn-iOS#620 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
google/GoogleSignIn-iOS#440 · 4 bình luận · 2 reaction ·
-
enhancement triage
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
google/GoogleSignIn-iOS#611 ·
-
Remove Cocoapods Support Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
google/GoogleSignIn-iOS#607 ·
Tất cả issue của google/GoogleSignIn-iOS
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
macOS下查找brew::ncurse失败 Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
noah-nuebling/mac-mouse-fix#2054 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
obsproject/obs-studio#13918 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
HigherOrderCO/Bend#842 · 2 bình luận ·