GrafeasGroup / buttercup

Our personal assistant in Discord
MIT License
5 stars 2 forks source link

Add `/footer` command #46

Open TimJentzsch opened 3 years ago

TimJentzsch commented 3 years ago

Add a /footer command that simply returns our footer. This might be useful for mobile transcribers who have difficulties copy & pasting from Reddit.

Options: None

bemxio commented 2 years ago

in which cog should it be written? i might take the issue and resolve it

TimJentzsch commented 2 years ago

This can be put in a separate footer cog if you'd like. You can make the setup analogous to the other cogs and then add it to the list in main.py. Let me know if you have any questions!

bemxio commented 2 years ago

i just have one question, should the footer be fetched from Reddit every time the command is used (with an additional cache of some sort), or should it be a pre-defined text somewhere in a config file?

TimJentzsch commented 2 years ago

I think it's better to define it in the localization file buttercup/strings/en_US.yaml. Examples on how to use this file are available in the other cogs.

bemxio commented 2 years ago

alright, i will work in my free time

bemxio commented 2 years ago

hmm, i tried to pick up on it, but when i tried to do poetry install, numpy was complaining about TypeError: CCompiler_spawn() got an unexpected keyword argument 'env' looked into it but couldn't find anything that would work @TimJentzsch is there a potential fix for that?

TimJentzsch commented 2 years ago

Hmm I am unable to reproduce the issue. Are you using the most recent version of Poetry?

TheLonelyGhost commented 2 years ago

This seems relevant: https://github.com/numpy/numpy/issues/17216

Short version is there seems to be some issue with Windows' compiler working with CPython and it's resolved by upgrading numpy to 1.22.x. That shouldn't be too difficult. 😄

Carrying discussion of this over to #170