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

NTV-607 : upgrade facebook sdk to latest for android 13 #1684

Merged

Conversation

hadia
Copy link
Contributor

@hadia hadia commented Sep 8, 2022

📲 What

Update the Facebook sdk to latest

🤔 Why

Login with facebook stopped

🛠 How

image

👀 See

Before
image
After

device-2022-09-08-174406.mp4
device-2022-09-08-174453.mp4

📋 QA

1- test login with facebook
2- test share facebook button on thanks activity apledgeeldge project

Story 📖

NTV-607

Signed-off-by: hadia <hadiamohamed.iti@gmail.com>
Signed-off-by: hadia <hadiamohamed.iti@gmail.com>
Signed-off-by: hadia <hadiamohamed.iti@gmail.com>
@hadia hadia added the migration Migration Java to Kotlin label Sep 8, 2022
@hadia hadia marked this pull request as ready for review September 8, 2022 15:46
app/build.gradle Outdated Show resolved Hide resolved
Signed-off-by: hadia <hadiamohamed.iti@gmail.com>
@hadia hadia requested a review from Arkariang September 8, 2022 16:16
@codecov
Copy link

codecov bot commented Sep 8, 2022

Codecov Report

Merging #1684 (119fa1a) into master (5db25e8) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1684      +/-   ##
============================================
- Coverage     78.60%   78.59%   -0.01%     
+ Complexity     1893     1892       -1     
============================================
  Files           359      359              
  Lines         17089    17089              
  Branches       2150     2150              
============================================
- Hits          13432    13431       -1     
  Misses         2404     2404              
- Partials       1253     1254       +1     
Impacted Files Coverage Δ
...pp/src/main/java/com/kickstarter/models/Message.kt 79.16% <0.00%> (-2.09%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@hadia hadia merged commit c94e7ab into master Sep 8, 2022
@hadia hadia deleted the hadia/NTV-607_upgrade_facebook_sdk_to_latest_for_android_13 branch September 8, 2022 17:33
Arkariang added a commit that referenced this pull request Sep 8, 2022
…ure/Android_13_Support

* 'master' of github.com:kickstarter/android-oss:
  NTV-607 : upgrade facebook sdk to latest for android 13 (#1684)
  - updated versions after release
  PAY-1893: Feature Flag Android PaymentSheet User Settings (#1682)
  WEB-665   [Android] When Facebook-only users log in to Kickstarter, require them to set a password (#1680)
  NTV-603: Mark as received UI inconsistent behaviour (#1681)

# Conflicts:
#	app/build.gradle
Arkariang added a commit that referenced this pull request Sep 12, 2022
…tin/PAY-1895-present-payment-sheet

* 'master' of github.com:kickstarter/android-oss:
  NTV-607 : upgrade facebook sdk to latest for android 13 (#1684)
  - updated versions after release
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migration Migration Java to Kotlin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants