Skip to content

docs: update migration notes#90

Merged
mogita merged 2 commits intomainfrom
docs/update-migration-notes
Mar 19, 2026
Merged

docs: update migration notes#90
mogita merged 2 commits intomainfrom
docs/update-migration-notes

Conversation

@mogita
Copy link
Copy Markdown
Contributor

@mogita mogita commented Mar 19, 2026

No description provided.

@mogita mogita requested a review from tbarbugli as a code owner March 19, 2026 16:04
Copilot AI review requested due to automatic review settings March 19, 2026 16:04
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:04 — with GitHub Actions Inactive
@mogita mogita had a problem deploying to feeds-enabled-shard March 19, 2026 16:04 — with GitHub Actions Failure
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:04 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:04 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:04 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:04 — with GitHub Actions Inactive
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the v3→v4 migration guide to better explain how v4 request struct serialization affects partial updates, especially around omitempty behavior.

Changes:

  • Updates the “After” example wire payload to illustrate that non-pointer slice/map fields are still serialized as null.
  • Expands the note explaining how nil vs empty slices/maps are encoded and how that impacts backend updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread MIGRATION_v3_to_v4.md
Comment thread MIGRATION_v3_to_v4.md Outdated
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:14 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:32 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:32 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:32 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:32 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:32 — with GitHub Actions Inactive
@mogita mogita temporarily deployed to feeds-enabled-shard March 19, 2026 16:32 — with GitHub Actions Inactive
@mogita mogita merged commit 1eac1e4 into main Mar 19, 2026
8 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.

2 participants