Skip to content

feat: expose cached on-chain subscription status - #1099

Merged
Chucks1093 merged 3 commits into
accesslayerorg:mainfrom
TekenujUi90:fix/issue-1079
Oct 27, 2026
Merged

Chucks1093 merged 3 commits into
accesslayerorg:mainfrom
TekenujUi90:fix/issue-1079

Conversation

@TekenujUi90

@TekenujUi90 TekenujUi90 commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

@accesslayerorg I updated this branch from main and resolved the merge conflicts. Please review and merge when ready.

Fixes #1079

@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@TekenujUi90 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

@TekenujUi90
TekenujUi90 force-pushed the fix/issue-1079 branch 2 times, most recently from 422f597 to 1407c5f Compare October 20, 2026 14:02
@TekenujUi90

Copy link
Copy Markdown
Contributor Author

@Chucks1093 I rebased this branch on current main and resolved the indexer conflict while preserving both the upstream invoice/TWAP/depth/whitelist/governance handlers and this PR’s subscription-created handler. Please review and merge when the checks pass.

@Chucks1093

Copy link
Copy Markdown
Member

Conflict

# Conflicts:
#	src/modules/indexer/indexer-pipeline.service.ts
@Chucks1093

Copy link
Copy Markdown
Member

⚠️ Merge Conflict Detected

This PR currently has a merge conflict and cannot be merged until it is resolved.

Please rebase or merge the latest changes from the base branch into your branch and push again.

Re-add the SubscriptionCreated handler on top of the upstream indexer
pipeline instead of replacing the file, so the newer poll, curve-update,
milestone, co-creator, reserve and tip handlers from main are kept.
@Chucks1093

Copy link
Copy Markdown
Member

✅ PR Ready for Review

Due to a recent malware incident affecting our repositories, all PRs are required to pass automated security checks before being reviewed for merging.

This PR has passed all automated checks:

  • ✅ Security scan — no malicious code detected
  • ✅ No merge conflicts
  • ✅ CI checks passed

Next step: Please share this PR link in our Telegram group so a maintainer can review and merge it: https://t.me/c/accesslayerorg/8514

@TekenujUi90

Copy link
Copy Markdown
Contributor Author

@Chucks1093 This PR is open, conflict-free, and its required checks are passing. Could you please review and merge it when ready? Thanks.

@Chucks1093
Chucks1093 merged commit d12ffc4 into accesslayerorg:main Oct 27, 2026
2 checks passed
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.

Build subscription status REST endpoint with expiry tracking

3 participants