Skip to content

Conversation

@freakboy3742
Copy link
Member

Provide an explicit link for the mailing list signup form, as a fallback when users have issues with the embedded form.

Fixes #607.

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

@freakboy3742 freakboy3742 added the preview Approved for an automated preview label Apr 10, 2025
@freakboy3742 freakboy3742 requested a review from mhsmith April 10, 2025 15:15
@github-actions
Copy link

github-actions bot commented Apr 10, 2025

Visit the preview URL for this PR (updated for commit b00f9c9):

https://beeware-org--pr610-members-link-tuclgwaq.web.app

(expires Wed, 23 Apr 2025 00:13:43 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: b0da44bc067e7d9a4255c77cb2c5fce572218cec

@mhsmith
Copy link
Member

mhsmith commented Apr 14, 2025

If a user fills in the the embedded form and it fails to submit, then there's a pretty good chance that they'll give up immediately rather than go to the trouble of filling it in again on the standalone page. And that's assuming they even notice the new link.

How about we remove the embedded form, and keep the link to the standalone form that we know always works? It has slightly better formatting as well.

@freakboy3742
Copy link
Member Author

If a user fills in the the embedded form and it fails to submit, then there's a pretty good chance that they'll give up immediately rather than go to the trouble of filling it in again on the standalone page. And that's assuming they even notice the new link.

Yeah - I'll freely admit I don't especially like the link option.

How about we remove the embedded form, and keep the link to the standalone form that we know always works? It has slightly better formatting as well.

It does - but it means linking off to a completely different site, and there's no obvious "return" path. My concern is the broader UX here. If it's just a link, then we're going to end up fighting the back button on a form submission; if we open a new window/tab, anyone with popup suppression might not see the form at all.

@mhsmith
Copy link
Member

mhsmith commented Apr 15, 2025

but it means linking off to a completely different site, and there's no obvious "return" path.

It does take you back to the BeeWare site afterwards, albeit via the pybee.org domain:

Screenshot 2025-04-15 at 08 15 13

We could change that to a more specific page like keep-informed.

The Mailchimp form also contains "Windows Phone", which I guess we should remove.

Apart from that, its UX seems fine to me.

@freakboy3742 freakboy3742 added preview Approved for an automated preview and removed preview Approved for an automated preview labels Apr 15, 2025
@freakboy3742
Copy link
Member Author

but it means linking off to a completely different site, and there's no obvious "return" path.

It does take you back to the BeeWare site afterwards, albeit via the pybee.org domain:

I've reconfigured the Mailchimp form to use beeware.org, and remove the Windows Phone option; given that there's a clear return path, I've replaced the entire community/keep-informed page with a redirect to the mailing list form.

@mhsmith
Copy link
Member

mhsmith commented Apr 15, 2025

Unfortunately it looks like this has removed the page entirely from the Community index page.

Also, it loses the link to Mastodon. I know there's a link in the footer of every page, but I don't know if you wanted it to be a bit more prominent in this context?

@freakboy3742
Copy link
Member Author

Ugh... I wasn't overly concerned about losing the Mastodon link - but I missed that it would lose the page on the Community index as well. Take 3...

@freakboy3742 freakboy3742 added preview Approved for an automated preview and removed preview Approved for an automated preview labels Apr 16, 2025
@mhsmith mhsmith merged commit 009dd5d into beeware:lektor Apr 16, 2025
2 checks passed
@freakboy3742 freakboy3742 deleted the members-link branch April 16, 2025 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Approved for an automated preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Newsletter Subscription Fails with Cross Origin Errors

2 participants