Skip to content

Conversation

@tejasbadadare
Copy link
Contributor

@tejasbadadare tejasbadadare commented Apr 30, 2025

Summary

Remove mention of gasConfig in the sub param validation function.

Rationale

Gas config is no longer part of a subscription's on-chain metadata, these docs are obsolete.

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@vercel
Copy link

vercel bot commented Apr 30, 2025

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

Name Status Preview Comments Updated (UTC)
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2025 5:39pm
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2025 5:39pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2025 5:39pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2025 5:39pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
api-reference ⬜️ Ignored (Inspect) Apr 30, 2025 5:39pm
component-library ⬜️ Ignored (Inspect) Apr 30, 2025 5:39pm

@tejasbadadare tejasbadadare merged commit 966dfa4 into main May 6, 2025
10 checks passed
@tejasbadadare tejasbadadare deleted the tb/pulse/update-validate-sub-params branch May 6, 2025 23:28
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.

3 participants