whats wrong with this

R

You need to provide a proper

MessagePage

MessagePage

MessagePage

MessagePage

R

RealShadowNova•1/30/21, 10:47 PM

export type MessagePage =
    | ((index: number, pages: MessagePage[], handler: PaginatedMessage) => Awaited<APIMessage | MessageOptions>)
    | APIMessage
    | MessageOptions

export type MessagePage =
    | ((index: number, pages: MessagePage[], handler: PaginatedMessage) => Awaited<APIMessage | MessageOptions>)
    | APIMessage
    | MessageOptions

export type MessagePage =
    | ((index: number, pages: MessagePage[], handler: PaginatedMessage) => Awaited<APIMessage | MessageOptions>)
    | APIMessage
    | MessageOptions

export type MessagePage =
    | ((index: number, pages: MessagePage[], handler: PaginatedMessage) => Awaited<APIMessage | MessageOptions>)
    | APIMessage
    | MessageOptions

This is the type for it.

R

RealShadowNova•1/30/21, 10:47 PM

I see you added

MessageOptions

MessageOptions

MessageOptions

MessageOptions to this Kyra.

R

RealShadowNova•1/30/21, 10:48 PM

I also want to point out that the you need to check whether the channel is a text or news channel instead of casting it. PM does not work in DMs.

V

vladdy•1/30/21, 10:48 PM

we still need to cast

V

vladdy•1/30/21, 10:49 PM

cuz message.channel is one of those three

MMagnaboy whats wrong with this

V

vladdy•1/30/21, 10:49 PM

whats the error 4head

M

MagnaboyOP•1/30/21, 10:49 PM

DiscordAPIError: Cannot send an empty message

DiscordAPIError: Cannot send an empty message

DiscordAPIError: Cannot send an empty message

DiscordAPIError: Cannot send an empty message

K

kyra•1/30/21, 10:49 PM

Laughs in GuildMessage type

R

RealShadowNova•1/30/21, 10:49 PM

^ exactly

V

vladdy•1/30/21, 10:50 PM

huh

R

RealShadowNova•1/30/21, 10:50 PM

I don't see why that wouldn't work. Are you sure it doesn't work?

V

vladdy•1/30/21, 10:50 PM

..i mean they get that error

V

vladdy•1/30/21, 10:50 PM

so

R

RealShadowNova•1/30/21, 10:50 PM

Can you log

display.pages

display.pages

display.pages

display.pages for me @magnaboy

R

RealShadowNova•1/30/21, 10:50 PM

or even better

display.messages

display.messages

display.messages

display.messages

V

vladdy•1/30/21, 10:51 PM

or even even better, both

R

RealShadowNova•1/30/21, 10:51 PM

after you add them

M

MagnaboyOP•1/30/21, 10:51 PM

i made it return

{embed: new MessageEm..}

{embed: new MessageEm..}

{embed: new MessageEm..}

{embed: new MessageEm..} instead of just

new MessageEm..

new MessageEm..

new MessageEm..

new MessageEm.. and seems to work now

R

RealShadowNova•1/30/21, 10:51 PM

hmm, ok

V

vladdy•1/30/21, 10:51 PM

huh

V

vladdy•1/30/21, 10:52 PM

so the callback is borked

R

RealShadowNova•1/30/21, 10:52 PM

show code Magna^

V

vladdy•1/30/21, 10:52 PM

talking about how we do the callback

V

vladdy•1/30/21, 10:52 PM

not Magna's code

R

RealShadowNova•1/30/21, 10:52 PM

How it compares to what didn't work

V

vladdy•1/30/21, 10:52 PM

probably display.addPage({ embed: new MessageEmbed })

R

RealShadowNova•1/30/21, 10:53 PM

M

MagnaboyOP•1/30/21, 10:53 PM

yea, i said

R

RealShadowNova•1/30/21, 10:53 PM

I just use

APIMessage

APIMessage

APIMessage

APIMessage so I guess I wouldn't know any better.

V

vladdy•1/30/21, 10:53 PM

wait you returned the object or?

V

vladdy•1/30/21, 10:53 PM

like did you still use a callback

M

MagnaboyOP•1/30/21, 10:53 PM

from

display.addPage(() => new MessageEmbed )

display.addPage(() => new MessageEmbed )

display.addPage(() => new MessageEmbed )

display.addPage(() => new MessageEmbed )
to

display.addPage(() => ({ embed: new MessageEmbed }))

display.addPage(() => ({ embed: new MessageEmbed }))

display.addPage(() => ({ embed: new MessageEmbed }))

display.addPage(() => ({ embed: new MessageEmbed }))

V

vladdy•1/30/21, 10:53 PM

oh

R

RealShadowNova•1/30/21, 10:53 PM

hmm

V

vladdy•1/30/21, 10:53 PM

R

RealShadowNova•1/30/21, 10:53 PM

both of those are supposed to work..

V

vladdy•1/30/21, 10:54 PM

blame kyra

R

RealShadowNova•1/30/21, 10:54 PM

sec

R

RealShadowNova•1/30/21, 10:54 PM

Yeah, broke the consistency with using APIMessage /s

RRealShadowNova Yeah, broke the consistency with using APIMessage /s

K

kyra•1/30/21, 10:55 PM

APIMessage is almost unusable

V

vladdy•1/30/21, 10:57 PM

i beg your pardon

V

vladdy•1/30/21, 10:57 PM

I use it

R

RealShadowNova•1/30/21, 10:57 PM

Yeah, it's extremely useful. I use it everywhere for PM

K

kyra•1/30/21, 10:57 PM

It doesn't support chaining

K

kyra•1/30/21, 10:57 PM

How's that usable?

V

vladdy•1/30/21, 10:57 PM

true

R

RealShadowNova•1/30/21, 10:58 PM

idgaf

K

kyra•1/30/21, 10:58 PM

v13 will eventually make APIMessage easier to use

R

RealShadowNova•1/30/21, 10:58 PM

Works for me

Similar Threads

Similar Threads

Similar Threads