Skip to content

fix: resolve issues #633-636 - #637

Merged
Chucks1093 merged 2 commits into
accesslayerorg:devfrom
ritaifeoluwa:fix/issues-633-634-635-636
Jul 27, 2026
Merged

fix: resolve issues #633-636#637
Chucks1093 merged 2 commits into
accesslayerorg:devfrom
ritaifeoluwa:fix/issues-633-634-635-636

Conversation

@ritaifeoluwa

Copy link
Copy Markdown
Contributor

Closes #633
Closes #634
Closes #635
Closes #636

What changed

Issue #633: Search clear resets URL and pagination

  • Added integration test verifying clearing search input removes search from URL query string and resets to page one

Issue #634: Error state component without onRetry

  • Added test confirming CreatorProfileErrorState renders without error when onRetry is not provided and retry button is absent

Issue #635: Sort dropdown initialization from URL

  • Added integration test loading page with ?sort=price-asc in URL, verifying dropdown initializes to correct option and initial fetch uses that sort

Issue #636: Structured cache invalidation log

  • Added debug-level [cache-invalidation] log in onSettled of useTradeMutation with fields: invalidated_keys, trigger, creator_id, invalidated_at
  • Log not emitted in test environment
  • Added unit tests verifying log emission and suppression

- Issue accesslayerorg#633: Add test for search clear resetting URL and pagination
- Issue accesslayerorg#634: Add test for error state component without onRetry
- Issue accesslayerorg#635: Add test for sort dropdown initialization from URL param
- Issue accesslayerorg#636: Add structured debug log for post-trade cache invalidation

Fixes accesslayerorg#633 accesslayerorg#634 accesslayerorg#635 accesslayerorg#636
@drips-wave

drips-wave Bot commented Jul 27, 2026

Copy link
Copy Markdown

@ritaifeoluwa Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

Fixes lint error in tradeCacheInvalidation.test.ts
@Chucks1093
Chucks1093 merged commit 3199177 into accesslayerorg:dev Jul 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment