Skip to content

Commit

Permalink
refactor: make data public in builders (#7486)
Browse files Browse the repository at this point in the history
  • Loading branch information
suneettipirneni committed Feb 20, 2022
1 parent 8dbd345 commit ba31203
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 2 deletions.
3 changes: 3 additions & 0 deletions packages/builders/src/components/ActionRow.ts
Expand Up @@ -15,6 +15,9 @@ export type ActionRowComponent = ButtonComponent | SelectMenuComponent;
export class ActionRow<T extends ActionRowComponent = ActionRowComponent> extends Component<
Omit<Partial<APIActionRowComponent<APIMessageComponent>> & { type: ComponentType.ActionRow }, 'components'>
> {
/**
* The components within this action row
*/
public readonly components: T[];

public constructor({ components, ...data }: Partial<APIActionRowComponent<APIMessageComponent>> = {}) {
Expand Down
2 changes: 1 addition & 1 deletion packages/builders/src/components/Component.ts
Expand Up @@ -19,7 +19,7 @@ export abstract class Component<
/**
* The API data associated with this component
*/
protected readonly data: DataType;
public readonly data: DataType;

public abstract toJSON(): APIMessageComponent;

Expand Down
Expand Up @@ -9,6 +9,9 @@ import isEqual from 'fast-deep-equal';
export class UnsafeSelectMenuComponent extends Component<
Partial<Omit<APISelectMenuComponent, 'options'>> & { type: ComponentType.SelectMenu }
> {
/**
* The options within this select menu
*/
public readonly options: UnsafeSelectMenuOption[];

public constructor(data?: Partial<APISelectMenuComponent>) {
Expand Down
2 changes: 1 addition & 1 deletion packages/builders/src/messages/embed/UnsafeEmbed.ts
Expand Up @@ -41,7 +41,7 @@ export interface EmbedImageData extends Omit<APIEmbedImage, 'proxy_url'> {
* Represents a non-validated embed in a message (image/video preview, rich embed, etc.)
*/
export class UnsafeEmbed implements Equatable<APIEmbed | UnsafeEmbed> {
protected data: APIEmbed;
public readonly data: APIEmbed;

public constructor(data: APIEmbed = {}) {
this.data = { ...data };
Expand Down

0 comments on commit ba31203

Please sign in to comment.