Skip to content

DMD-2389 Add no artifact to AIA#63

Open
MarioHi1 wants to merge 1 commit into
mainfrom
DMD-2389
Open

DMD-2389 Add no artifact to AIA#63
MarioHi1 wants to merge 1 commit into
mainfrom
DMD-2389

Conversation

@MarioHi1
Copy link
Copy Markdown
Contributor

@MarioHi1 MarioHi1 commented May 4, 2026

No description provided.

Copy link
Copy Markdown
Contributor

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

This PR extends the Advanced Illustration Area schema with a new noArtifact flag and regenerates the language-specific protobuf artifacts that expose that field to consumers of this repository.

Changes:

  • Added noArtifact to ProtoAdvancedIllustrationArea in the source .proto.
  • Regenerated the aggregated proto plus Go, JS/TS, and C# artifacts to include the new field.
  • Updated generated accessors/serialization code so the field is encoded and decoded across supported outputs.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/DocumentElements/documentelements.advanced_illustration_area.proto Adds the new source-of-truth field to the AIA message.
build/docframe.proto Regenerated combined proto including the new field.
docframepb/docframe.pb.go Regenerated Go bindings/accessors for the new field.
build/Docframe.js Regenerated JavaScript protobuf runtime support for the new field.
build/Docframe.d.ts Regenerated TypeScript typings for the new field.
build/Docframe.cs Regenerated C# protobuf bindings for the new field.

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

ProtoAdvancedIllustrationAreaTextFlowType textFlow = 9;
float rotation = 10;
repeated string comChannelUUIDs = 11;
bool noArtifact = 12;
Comment thread docframepb/docframe.pb.go
// versions:
// protoc-gen-go v1.36.11
// protoc v7.34.1
// protoc v6.33.4
@MarioHi1 MarioHi1 assigned nicojoech and unassigned podsi May 7, 2026
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.

4 participants