Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

help: using a pastebin should be optional #1212

Closed
kkirsche opened this issue Aug 4, 2017 · 11 comments · Fixed by #2332
Closed

help: using a pastebin should be optional #1212

kkirsche opened this issue Aug 4, 2017 · 11 comments · Fixed by #2332
Milestone

Comments

@kkirsche
Copy link

kkirsche commented Aug 4, 2017

Not all environments are public environments where it is appropriate to post commands publicly. It would be extremely helpful if this behavior (print to channel or gist) was configurable.

@dgw
Copy link
Member

dgw commented Aug 4, 2017

Also additional options, like outputting to a text or HTML file on the local filesystem instead, would be useful. Maybe I should just bite the bullet and implement all that.

@dgw
Copy link
Member

dgw commented Mar 18, 2018

As an aside, this issue might be obsolete. See #1254.

@dgw
Copy link
Member

dgw commented Mar 28, 2018

Switched to a different pastebin provider for the time being. This will stick around as a feature request.

@dgw dgw changed the title .help option of using a gist should be configurable .help option of using a pastebin should be configurable Mar 28, 2018
@dgw dgw added this to the 7.0.0 milestone Dec 1, 2018
@Exirel
Copy link
Contributor

Exirel commented Sep 26, 2019

I feel like this has been fixed by #1451

@dgw
Copy link
Member

dgw commented Sep 26, 2019

@Exirel Nope. We still want to add locally-hosted HTML or something that doesn't touch a pastebin at all. Choosing a pastebin was a separate issue, #1416, as confusing as the similar issue titles might be.

@dgw dgw reopened this Sep 26, 2019
@dgw dgw changed the title .help option of using a pastebin should be configurable help: using a pastebin should be optional Sep 26, 2019
@dgw dgw modified the milestones: 7.0.0, 7.1.0 Nov 16, 2019
@dgw
Copy link
Member

dgw commented Nov 16, 2019

Punting this. The implementation should add an option without changing the existing behavior if not configured, so no rush to get this into the next major version.

@Exirel
Copy link
Contributor

Exirel commented Jan 10, 2020

The more I think about that one, the more I'd like to create a separated package for the help plugin, and to have a dedicated [help] config section. I've plenty of ideas for that kind of plugins, and I think it would help (...) a lot to be able to update such plugin whenever a paste service become unavailable, or when they change their interface, and so on.

For instance, I'd love to allow third-party devs to implement their own help publishing backend, so whenever someone have a new use-case that is solely for them and their group, they could do it more easily.

Also, I'd use it as an excuse to push Sopel's internals to always be better - as one may say "eat your own food".

@dgw
Copy link
Member

dgw commented Jan 10, 2020

Aren't we already pulling a bunch more stuff out of core for 8.0? help can be part of that.

@Exirel
Copy link
Contributor

Exirel commented Jan 10, 2020

Yes, but I'd love to start with sopel-help for Sopel 7.1: it's easy to do, and since almost everyone use it from time to time, if anything goes wrong it won't be silent.

Also, "help" is another name for "documentation". 😛

@dgw
Copy link
Member

dgw commented Jan 10, 2020

Feel free to play around in the new sopel-help repo, @Exirel. Whatever you come up with in there, the only real question is whether it takes effect in 7.1 or 8.0.

@dgw
Copy link
Member

dgw commented Apr 12, 2020

We'll consider this fixed by sopel-irc/sopel-help, which includes several pastebins, adds local HTML output (to be served by the user's chosen external HTTPd), and resurrected direct-to-IRC output as well.

Since the new plugin only supports Python 3.5+, I'll mark this for Sopel 8.0 (which will drop support for older Python versions) and note it prominently in the release notes for that version, then close the issue.

@dgw dgw modified the milestones: 7.1.0, 8.0.0 Apr 12, 2020
@dgw dgw added Replaced Superseded by a newer PR and removed Patches Welcome labels Apr 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants