feat!: delete Link state views#68
Conversation
BREAKING CHANGE: Removes the LinkToken and StaticLinkToken state views from the public API.
|
👋 jkongie, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
Removes the LinkToken and StaticLinkToken “state view” types from the module’s public API (breaking change), along with their associated view generation helpers and tests.
Changes:
- Deleted the
LinkTokenView/GenerateLinkTokenViewimplementation. - Deleted the
StaticLinkTokenView/GenerateStaticLinkTokenViewimplementation. - Removed unit/integration tests that exercised those view generators.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/contract/link/view/v10/static_link_token.go | Removed the StaticLinkTokenView type and generator from the public API. |
| pkg/contract/link/view/v10/static_link_token_test.go | Removed tests covering GenerateStaticLinkTokenView. |
| pkg/contract/link/view/v10/link_token.go | Removed the LinkTokenView type and generator from the public API. |
| pkg/contract/link/view/v10/link_token_test.go | Removed tests covering GenerateLinkTokenView (including zk-specific test). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|





BREAKING CHANGE: Removes the LinkToken and StaticLinkToken state views from the public API.