Skip to content

fix(tapd): correct field mapping for TapdScopeConfig#8399

Merged
klesh merged 1 commit into
mainfrom
fix-tapd
Apr 23, 2025
Merged

fix(tapd): correct field mapping for TapdScopeConfig#8399
klesh merged 1 commit into
mainfrom
fix-tapd

Conversation

@narrowizard
Copy link
Copy Markdown
Collaborator

  • Update the mapstructure tag for ScopeConfig field from "pageSize" to "scopeConfig"
  • This change ensures proper deserialization of ScopeConfig data in TapdOptions struct

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

What does this PR do?

Does this close any open issues?

Closes xx

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

- Update the mapstructure tag for ScopeConfig field from "pageSize" to "scopeConfig"
- This change ensures proper deserialization of ScopeConfig data in TapdOptions struct
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. component/plugins This issue or PR relates to plugins pr-type/bug-fix This PR fixes a bug labels Apr 23, 2025
@klesh klesh merged commit 53dab91 into main Apr 23, 2025
13 checks passed
@klesh klesh deleted the fix-tapd branch April 23, 2025 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/plugins This issue or PR relates to plugins pr-type/bug-fix This PR fixes a bug size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants