Skip to content

Conversation

@dinosoeren
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dinosoeren dinosoeren changed the title Drop rates API Offer details API Nov 14, 2025
},
(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {description: "Optional region filter using ISO-3166-1 alpha-2 country code (e.g., 'US', 'EU', 'JP')"}
];
optional string language = 4 [

Choose a reason for hiding this comment

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

we may need to use the 4 letter code here i think to differentiate between regional languages like es-ES vs es-MX

];
optional string background_style = 4 [(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {description: "Optional background styling for metadata section"}];
}
OfferMetadata metadata = 3 [

Choose a reason for hiding this comment

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

as we discussed in our last 1:1 it looks like we also need to add in an array of some sort to represent the list of sections in the metadata and their contents. Note that in some cases text is uppercased vs not so we need a way to configure that as well

Screenshot 2025-11-20 at 12 00 10 PM

}
};
LocalizableText text = 1 [(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {description: "Header text content"}];
optional uint32 relative_width = 2 [

Choose a reason for hiding this comment

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

are we going to validate that the widths of all the headers add to 100? if not how do you envision we handle that?

Choose a reason for hiding this comment

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

also if not provided, should we assume the space is evenly distributed then?

];
}

message Tab {

Choose a reason for hiding this comment

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

this needs to also have a tab title right? (LocalizableText)

(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {description: "Optional URL to icon image for the cell"}
];
optional string background_style = 4 [(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {description: "Optional CSS-style background styling for the cell"}];
optional string text_style = 5 [(grpc.gateway.protoc_gen_openapiv2.options.openapiv2_field) = {description: "Optional CSS-style text styling for the cell"}];

Choose a reason for hiding this comment

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

what is an example format of how you see this being used? there are different text properties that could be configured like text-transform​, color​, etc. would those be combined into one string here or should this be for a more specific text styling?

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.

3 participants