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

@W-16797266: [TRUST] Ensure all WorkManager Jobs have an associated tag or unique name #2629

Conversation

JohnsonEricAtSalesforce
Copy link
Contributor

@JohnsonEricAtSalesforce JohnsonEricAtSalesforce commented Oct 18, 2024

🤘🏻 Ready For Review! 🎸

This simple Trust update ensures the Android Background Tasks library will not be used to create duplicate work.

@@ -49,7 +49,6 @@ import com.salesforce.androidsdk.push.PushMessaging.getRegistrationId
import com.salesforce.androidsdk.push.PushMessaging.setRegistrationId
import com.salesforce.androidsdk.push.PushMessaging.setRegistrationInfo
import com.salesforce.androidsdk.push.PushNotificationsRegistrationChangeWorker.PushNotificationsRegistrationAction
import com.salesforce.androidsdk.push.PushNotificationsRegistrationChangeWorker.PushNotificationsRegistrationAction.Deregister
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I confirmed this was an unused import. It wasn't used in the logic on dev in this source. It is used elsewhere, though.

@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for SalesforceSDK

Generated by 🚫 Danger

@mobilesdk-bot
Copy link

1 Error
🚫 Tests have failed, see below for more information.
1 Warning
⚠️ No Lint Results.

Tests:

Name Classname Time
testCleanupRegexpFaster com.salesforce.androidsdk.smartstore.store.SmartSqlTest 3.259

Tests results for SmartStore

Generated by 🚫 Danger

@mobilesdk-bot
Copy link

1 Warning
⚠️ No Lint Results.

Tests results for SalesforceReact

Generated by 🚫 Danger

@JohnsonEricAtSalesforce JohnsonEricAtSalesforce merged commit ebe43e3 into forcedotcom:dev Oct 19, 2024
5 of 9 checks passed
@JohnsonEricAtSalesforce JohnsonEricAtSalesforce deleted the feature/w-16797266_trust-ensure-all-workmanager-jobs-have-an-associated-tag-or-unique-name branch October 19, 2024 00:21
Copy link
Contributor

@wmathurin wmathurin left a comment

Choose a reason for hiding this comment

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

👍

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.

4 participants