Skip to content

Commit

Permalink
Mark proto fields as optional
Browse files Browse the repository at this point in the history
Allows future versions to deprecate and remove fields in a two step process.

For more details see #465 (comment)
  • Loading branch information
mxinden committed Jan 18, 2023
1 parent e384839 commit b5d82c4
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions webrtc/browser-to-browser.md
Original file line number Diff line number Diff line change
Expand Up @@ -78,9 +78,8 @@ message Message {
CANDIDATE = 2;
}
// TODO: Consider removal of `required` for future compatibility.
required Type type = 1;
required string data = 2;
optional Type type = 1;
optional string data = 2;
}
```

Expand Down

0 comments on commit b5d82c4

Please sign in to comment.