INN / umbrella-sfpublicpress

San Francisco Public Press
https://sfpublicpress.org/
GNU General Public License v2.0
1 stars 4 forks source link

Update largo_donate_button to use Pico class #90

Closed joshdarby closed 4 years ago

joshdarby commented 4 years ago

Changes

This pull request makes the following changes:

Will look like this on prod:

Screen Shot 2020-06-22 at 10 31 54 AM

Why

For #66

Testing/Questions

Features that this PR affects:

Questions that need to be answered before merging:

Steps to test this PR:

  1. Make sure nothing is broken
benlk commented 4 years ago

I doubt anything will appear when the button is clicked on your local environment unless you have the Pico plugin set up. If you want to try setting it up, install the plugin and log in to Pico with the creds in 1Pass.

The Pico plugin is installed and active and logged in, but I'm getting CORS and 422 errors in the browser console, and the popup never opens.

Should this link keep the current href as a fallback option?

How come the Largo Donate widget doesn't use a filterable version of largo_donate_button?

Looks like it's because that widget allows setting arbitrary links in the widget backend. largo_donate_button() has no arguments allowing text or link to be set, but we could add that in a future version of Largo, with fallback params being the current database-fetched items. Is it worth creating a ticket for that?