Merge r2.8.7 with code 299 from develop to master

[jaymengxy]

Related Issue

Closes #???

Summary of Changes

Need Regression Testing

• Yes
• No

Risk Assessment

• Low
• Medium
• High

Additional Notes

Screenshots (if applicable)

[github-actions]

PR Summary

Fixed an issue where duplicate tokens could appear in the token list display by adding distinct filtering by contract ID. This ensures each token appears only once in the list, improving the user experience and data accuracy. Also bumped version code to 299.

Changes

File	Summary
app/src/main/java/com/flowfoundation/wallet/manager/token/FungibleTokenListManager.kt	Modified token list management to prevent duplicates by adding distinctBy { it.contractId() } when loading from cache, initial provider load, and list updates. This ensures unique tokens based on contract ID.
gradle.properties	Incremented version code from 298 to 299 for release r2.8.7.

---

autogenerated by presubmit.ai

[github-actions]

🚨 Pull request needs attention.

Review Summary

Commits Considered (2)

• 2dd00eb: Merge pull request #1173 from onflow/fix_token_list

fix: duplicate token list

• c82f77f: fix: duplicate token list

Files Processed (2)

• app/src/main/java/com/flowfoundation/wallet/manager/token/FungibleTokenListManager.kt (3 hunks)
• gradle.properties (1 hunk)

Actionable Comments (1)

• app/src/main/java/com/flowfoundation/wallet/manager/token/FungibleTokenListManager.kt [142-142]

  possible issue: "Potential null pointer issue in distinct operation"

Skipped Comments (1)

• app/src/main/java/com/flowfoundation/wallet/manager/token/FungibleTokenListManager.kt [178-178]

  performance: "Missing input validation for potentially large lists"

[github-actions]

Consider handling potential null values returned by contractId(). If contractId() can return null, the distinctBy operation might not work as expected and could lead to unexpected behavior. You might want to add a null check or use distinctBy { it.contractId() ?: "" } to handle this case safely.