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

Remove Gratipay integration - gratipay has shut down #128

Closed
wants to merge 1 commit into from

Conversation

joealcorn
Copy link

Gratipay shut down at the end of 2017. It's a shame because open source sustainability is an important topic. Perhaps instead of removing donate.html, we could extend it to support patreon and open collective? (are there other services?)

@bitprophet
Copy link
Collaborator

Weird, I thought this already got nixed but I must be misremembering or recalling their old name-change from Gittip. Thanks for the input!

I like the idea of changing it to something else instead of straight removal, especially since the template name and some of the language is sufficiently generic. May leave this open until I or others have time to really scan the horizon for what's best.

Tho...probably just want to make the setting names & everything else extra-generic so users can plug in whatever they want? May require expecting users to modify their own template, since these sorts of buttons are usually custom HTML tags (like the Gratipay one was.)

@bitprophet
Copy link
Collaborator

Consensus at sprints today is that we should just move ahead with merging this and worry about re-adding a generic donation.html later.

My main question now (and one that has been long-standing and long-avoided) is how to release this sort of VERY breaking change while ALSO distributing non-breaking changes. We're pinned at >0.7,<0.8 in Sphinx's setup.py, so I've been just incrementing the bugfix number, but that won't fly for this. So we should just pop out a 1.0 probably, have Sphinx eventually require that version (or just be looser, which may be better...I no longer recall why we wanted to tighten it so much) and try to iterate more normally.

@bitprophet
Copy link
Collaborator

I'll think about this some more and poke around, hopefully we'll figure something out by end of sprints. May go bother any Sphinx people who are here at PyCon to get their opinion.

@bitprophet bitprophet mentioned this pull request May 15, 2018
4 tasks
@bitprophet
Copy link
Collaborator

bitprophet commented May 15, 2018

Doesn't seem like Sphinx is sprinting this year but I might still put out a call to see if principals happen to be here anyway.

That said I triple checked my old commit that pinned us, and thanks to my habit of detailed commit messages, I can confirm the main point was to prevent a loose pin from meaning users incidentally upgraded past incompatible versions: sphinx-doc/sphinx#1759

So I think what this means is:

  • Another PR to sphinx master to change the pin from >=0.7,<0.8 to >=1.0,<2.0, which would hopefully then get included in Sphinx uh...1.8
  • Merge this
  • Seek out any other outstanding backwards incompat changes and consider them too
  • Make 100% sure we have a useful 0.7.x branch created for releasing backwards compatible changes to users of Sphinx<1.8
  • Bump us to 1.0.0
  • Update my other Sphinx plugins to work with Sphinx 1.7 (it broke Releases for example)
  • Eagerly await Sphinx 1.8

@joealcorn
Copy link
Author

how to release this sort of VERY breaking change

It's breaking in the sense that the functionality no longer exists, but having those options in your config shouldn't break the build (as far as I'm aware?) so is it really "breaking"?

@bitprophet
Copy link
Collaborator

bitprophet commented May 15, 2018

@joealcorn Sadly, it is breaking, I was not happy to discover that. I tested what a user who's currently setting gratipay_user = 'foo' would experience with the PR merged, and Sphinx goes kaboom, because setting an option which the theme doesn't account for, is considered an error.

Similarly, a user referencing donation.html in their sidebar config will encounter an error when that file is removed.

So at that point we're really facing down just jumping straight to #132, though even if a nice generic patch for that appeared today we'd need to make sure it still didn't look fugly for anyone who previously had the gratipay version. (And we'd still have to keep the gratipay and gittip user options around, etc. Gotta rip that bandaid off sometime...)

@bitprophet
Copy link
Collaborator

bitprophet commented Oct 2, 2018

Returning to this while handling #142, #143 - I don't have the bandwidth to handle the planned jump to 1.0 right now, and since we'd probably have wanted to do some version of this anyways, I think I'm just gonna:

  • Remove the gratipay-related output in donation.html, while leaving the option in place in the conf file (& documented as deprecated) to avoid abovementioned Sphinx errors.
    • Anyone who hasn't yet removed the setting from their own options will have the sidebar content mysteriously disappear when they upgrade...
    • But since the current output doesn't do anything/go anywhere (it uses http://gttp.co/v1.js which is a 404 today) I don't think that's an actual problem anyone can possibly complain about? Unless they're doing a lot of custom styling, I guess. Seems like an edge case & if they're that customized, they'll know to look at our changelog when they upgrade...
  • Inject some variant of the code from Add donate_url and opencollective options to add donate buttons to the sidebar #143 in its place, namely an arbitrary donation related link/badge.
  • Release an 0.7.x as I've been doing.

@bitprophet bitprophet closed this in d7c124c Oct 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants