fix: getting message pinss

[CyberL1]

Before:

After:

[MaddyUnderStars]

Imo it would be better to remove the pinned column entirely and just use this date column to check if it's pinned

[TheArcaneBrony]

Wouldnt this require changes in gateway etc?

[TheArcaneBrony]

That was supposed to be a thread, oops

[s074]

You can add a computed property so we don't have to do a null check on pinned_at every time we want to check if it's pinned:

Suggested change

	get pinned(): boolean {
	return this.pinned_at != null
	}

and then we also have to add it to the toJSON() method since getters aren't serialized when we have to do JSON.stringify on the object:

toJSON(): Message {
		return {
			...this,
			author_id: undefined,
			member_id: undefined,
			webhook_id: this.webhook_id ?? undefined,
			application_id: undefined,

			nonce: this.nonce ?? undefined,
			tts: this.tts ?? false,
			guild: this.guild ?? undefined,
			webhook: this.webhook ?? undefined,
			interaction: this.interaction ?? undefined,
			reactions: this.reactions ?? undefined,
			sticker_items: this.sticker_items ?? undefined,
			message_reference: this.message_reference ?? undefined,
			author: {
				...(this.author?.toPublicUser() ?? undefined),
				// Webhooks
				username: this.username ?? this.author?.username,
				avatar: this.avatar ?? this.author?.avatar,
			},
			activity: this.activity ?? undefined,
			application: this.application ?? undefined,
			components: this.components ?? undefined,
			poll: this.poll ?? undefined,
			content: this.content ?? "",
                        pinned: this.pinned,
		};
	}

[s074]

This PR looks good to me, only missing migrating old pinned messages to the new format. Probably just fetch all the messages that are pinned, then set the pinned_at timestamp to current time since we don't know when it was pinned, and finally drop the old pinned column