availability of .latex command about sir-lancebot HOT 7 CLOSED

It should go into the LATEX_ALLOWED_CHANNNELS, since we're talking about allowing the latex command, that'll automatically include it in the override later on.

The help channel isn't in the Channels class, so you would need to add it as well.

from sir-lancebot.

This sounds fine to me, we can always restrict it again if there's an issue.

from sir-lancebot.

I could help with this issue. I'd just need clarification on the finalized approach:

1. Allow in all channels except X,Y,Z.
2. Add X,Y,Z to the LATEX_ALLOWED_CHANNELS.

from sir-lancebot.

I could help with this issue. I'd just need clarification on the finalized approach:

1. Allow in all channels except X,Y,Z.
2. Add X,Y,Z to the LATEX_ALLOWED_CHANNELS.

Its the reverse, you add the the channels which are allowed in LATEX_ALLOWED_CHANNELS. So if channel XYZ isn't allowed you don't add it to the LATEX_ALLOWED_CHANNELS. Only allowed channels are to be added to that variable.

from sir-lancebot.

@matiaslagoevia I can assign you if you want to work on this.

The change should be as simple as adding the python help-channel id to the list of allowed channels, just like in the snippet attached to the issue

from sir-lancebot.

@matiaslagoevia I can assign you if you want to work on this.

The change should be as simple as adding the python help-channel id to the list of allowed channels, just like in the snippet attached to the issue

Thanks! I wasn't sure since the OP mentioned two different approaches in the description.

I'd be inclined to add it here, instead of here.

Also, I'm not sure what channel in the list here would be the appropriate one (or if I should add it to that list as well).

from sir-lancebot.

Got it, I have enough context to tackle the issue now. Thank you!

from sir-lancebot.