feat: ✨ Add spamFilteringTx in LLD

[mcayuelas-ledger]

✅ Checklist

• npx changeset was attached.
• Covered by automatic tests.
• Impact of the changes:

1. Transaction Filtering:
   Implementation of a new feature to filter transactions involving spam NFTs. This ensures that only valid and relevant transactions are considered within the system.

2. Whitelist and Enhancement of HiddenCollection:
   Addition of a whitelist to control the addresses and accounts authorized to interact with the system, enhancing the security and reliability of processed transactions.
   Improvements made to the HiddenCollection object to optimize the management and storage of hidden NFT collections.

3. Automatic Synchronization:
   Implementation of automatic synchronization every 6 hours to refresh and filter NFTs via the Simplehash API. This ensures that the data remains up-to-date and relevant.

4. Refresh on Account Addition:
   When new accounts or addresses are added, the filtering request is automatically re-triggered. This guarantees that new entries are immediately taken into account in the filtering process.

5. Add loading "Show more collections" to avoid load everything fully

📝 Description

0 - News
1 - Tests
2 - Feature

0 - What's new

• Rework some components using useNftGalleryFilter
• New generic hook useNftCollections used instead in each file that was using useNftGalleryFilter
• HiddenCollection and Whitelist NFTs

1 - TESTS Feature with SimpleHashTool + TanStackQuery

Test if we have more than 20 address if check is done on other groups

Screen.Recording.2024-09-25.at.14.41.07.mov

Adding a new account

Screen.Recording.2024-09-25.at.15.21.38.mov

Test FF

Screen.Recording.2024-09-25.at.16.22.37.mov

2 - Spam Filtering

• ADD in HiddenCollection
• add new account
• Filter Spam TX
• Show more

Screen.Recording.2024-10-28.at.16.21.13.mov

Screen.Recording.2024-10-28.at.16.22.19.mov

❓ Context

• JIRA or GitHub link: LIVE-14035

---

🧐 Checklist for the PR Reviewers

• The code aligns with the requirements described in the linked JIRA or GitHub issue.
• The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
• There are no undocumented trade-offs, technical debt, or maintainability issues.
• The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
• Any new dependencies have been justified and documented.
• Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
web-tools	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 30, 2024 8:41am

4 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
ledger-live-docs	⬜️ Ignored (Inspect)	Visit Preview		Oct 30, 2024 8:41am
ledger-live-github-bot	⬜️ Ignored (Inspect)	Visit Preview		Oct 30, 2024 8:41am
native-ui-storybook	⬜️ Ignored (Inspect)	Visit Preview		Oct 30, 2024 8:41am
react-ui-storybook	⬜️ Ignored (Inspect)	Visit Preview		Oct 30, 2024 8:41am

[themooneer]

i've added an optional suggestion. but it's a nice and clean work 👏

[thesan]

Should this be defined here or imported from libs/ledger-live-common/src/market/utils/timers.ts ? Because I don't see HALF_DAY imported anywhere.

[mcayuelas-ledger]

perhaps we could consider a file with all the timers, as this one is specific to the market, for example ? wdyt?

[thesan]

Yes it's good too but what I suggested here here was either:

• To import ONE_DAY and HALF_DAY from ledger-live-common/src/market/utils/timers

• Or to remove HALF_DAY from ledger-live-common/src/market/utils/timers and remove the export from the declaration:

  Suggested change

  	export const ONE_DAY = 24 * 60 * 60 * 1000;
  	export const HALF_DAY = ONE_DAY / 2;
  	const ONE_DAY = 24 * 60 * 60 * 1000;
  	const HALF_DAY = ONE_DAY / 2;

It's a detail but since HALF_DAY is not imported anywhere it looks like extra unused code right now. Unless of course if you're planning to use it later.

[themooneer]

Suggestion it("should map a multiple chains")

[KVNLS]

It works well! Well done!