Sapphire - Imagine a Framework•5y ago

Yeah, broke the consistency with using APIMessage /s

RRealShadowNova Yeah, broke the consistency with using APIMessage /s

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

APIMessage is almost unusable

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

i beg your pardon

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

I use it

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

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

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

It doesn't support chaining

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

How's that usable?

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

true

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

idgaf

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

v13 will eventually make APIMessage easier to use

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

Works for me

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

But that's not now

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

but I do agree making APIMessage a builder would be useful.

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

kyra are you opposed to having the behaviour of, the bot not removing the arrow reactions when you click on them, so to cycle through pages you add/remove/add/remove, in other words you dont have to wait for the bot to remove the reaction, and the bot doesnt waste api calls? it feels a lot more fluid to me

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

oh gosh

RealShadowNovaOP•1/30/21, 10:59 PM

I had this discussion before with Kyra, he said that he doesn't want that type of behavior. This was when I was making the util btw.

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

Nova

RealShadowNovaOP•1/30/21, 10:59 PM

I don't know about now.

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

You might have been imagining things

RealShadowNovaOP•1/30/21, 11:00 PM

MMagnaboy kyra are you opposed to having the behaviour of, the bot not removing the arrow ...

vladdy•1/30/21, 11:00 PM

the good news is that you can overwrite the function that does it! But yeah, I concur, it would be nicer if, when we don't have manage_message perms, we let you unreact to switch pages

kyra•1/30/21, 11:00 PM

tfw you're time travelling

unknown.png3.08KB

RealShadowNovaOP•1/30/21, 11:00 PM

yes.

RealShadowNovaOP•1/30/21, 11:00 PM

I assume it's midnight for u

Magnaboy•1/30/21, 11:01 PM

so vlad you think its not better, only as a fallback?

vladdy•1/30/21, 11:01 PM

the ratelimit isn't harsh enough

vladdy•1/30/21, 11:02 PM

If you really want to, it should be an option

vladdy•1/30/21, 11:02 PM

defaulting to true for.. non dm channels unless we lack manage_messages

RealShadowNovaOP•1/30/21, 11:02 PM

You could actually extend PM and make a separate class for it

RealShadowNovaOP•1/30/21, 11:03 PM

That way it doesn't bloat the one class.

Magnaboy•1/30/21, 11:05 PM

as a fallback would definitely not be bloat

RealShadowNovaOP•1/30/21, 11:06 PM

Also, Kyra I was thinking about a refreshPagesrefreshPages method for PM cause I modify the pages within actions (handler.pages[handler.index] = ...handler.pages[handler.index] = ...) and I have to then run setPages(handler.pages)setPages(handler.pages) to refresh the pages. Also, maybe need a method to edit the current page even if the index is the same.

vladdy•1/30/21, 11:06 PM

I concur with Magna

Magnaboy•1/30/21, 11:07 PM

can i pass a option to disable the first/last reactions?

unknown.png1.52KB

RealShadowNovaOP•1/30/21, 11:07 PM

actions are up to you, we just provide default ones.

vladdy•1/30/21, 11:07 PM

you'd have to set the actions to whatever floats your boat

vladdy•1/30/21, 11:08 PM

I lowkey copied them all and just dropped those

vladdy•1/30/21, 11:08 PM

Also, you can technically alter the defaults

vladdy•1/30/21, 11:08 PM

locally

RealShadowNovaOP•1/30/21, 11:08 PM

So if you want to filter some out just do

new PaginatedMessage({ actions: PaginatedMessage.defaultActions.filter(action => ['emoji1', 'emoji2'].includes(action.id)) })

new PaginatedMessage({ actions: PaginatedMessage.defaultActions.filter(action => ['emoji1', 'emoji2'].includes(action.id)) })

Magnaboy•1/30/21, 11:11 PM

oh noice making actions looks easy

RealShadowNovaOP•1/30/21, 11:11 PM

They really are tbh

vladdy•1/30/21, 11:12 PM

its called

F l e x i b i l i t y

kyra•1/30/21, 11:13 PM

I remember when Nova was calling me out because I was suggesting him a complicated thing

RealShadowNovaOP•1/30/21, 11:13 PM

I mean, my bot uses PM a lot of places and are extremely easy to use and adjust per use-case

RealShadowNovaOP•1/30/21, 11:13 PM

shhhh, we don't talk about \s

kyra•1/30/21, 11:13 PM

Magnaboy•1/31/21, 4:32 PM

how should i write code if i want to pick an arg which i want to be either a textchannel, or a specific string?

e.g. +modlogs #modlogs+modlogs #modlogs or +modlogs off+modlogs off

vladdy•1/31/21, 4:33 PM

well

vladdy•1/31/21, 4:33 PM

first try to see if a channel matches

vladdy•1/31/21, 4:33 PM

if that doesn't match

Yeah, broke the consistency with using APIMessage /s

Similar Threads

Similar Threads

Similar Threads