Skip to content

feat: Introducing publisher tools #129

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

lengyel-arpad85
Copy link

No description provided.

@lengyel-arpad85 lengyel-arpad85 self-assigned this May 19, 2025
@lengyel-arpad85 lengyel-arpad85 marked this pull request as draft May 19, 2025 09:16
@lengyel-arpad85 lengyel-arpad85 marked this pull request as ready for review May 19, 2025 14:45
- **Button**
- **Exclusive Content**

Each element is fully customizable via a visual admin interface where users can configure text, colors, fonts, and more - with live preview support. *No coding skills needed.*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Each element is fully customizable via a visual admin interface where users can configure text, colors, fonts, and more - with live preview support. *No coding skills needed.*
Each element is fully customizable via a visual admin interface where creators can configure text, colors, fonts, and more - with live preview support. *No coding skills needed.*

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I forgot about that. We need to be clear who the users are. At the beginning of the post, you have "...designed to promote Web Monetization as a model for users to support their websites" where users here are site visitors. Here users means content owners, publishers, site owners, whatever we want to call them. "Users" is used a few times throughout the post.


This approach lowers the technical barrier and ensures creators can focus on their content - not website maintenance.

## Ensuring Wallet Ownership and Security
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The previous section mentions "automatically pulled based on the wallet address" kind of in passing. We should add something that's more explicit. Also, I wonder if a screenshot would be helpful. It might be too off topic, but I'm throwing the idea out there anyway.

You don't have to go with this, but here's a suggestion for the Ensuring Wallet Ownership and Security section:

Creators are asked to confirm they own their wallets before a script tag is generated. Without ownership confirmation, their customizations are not saved on the server. The validation steps help to maintain integrity, protect wallet owners, and ensure only the rightful owner of a wallet can update the configuration.

In the future, a more streamlined wallet verification mechanism will replace this process.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small, annoying thing, but can this image be updated so that Web Monetization is capitalized?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, capitalize Web Monetization

@JoblersTune
Copy link
Contributor

@lengyel-arpad85 it looks good to me (I see the code styling is broken but that's nothing to do with you and I'll need to make a new PR to fix that).

The only thing is ideally we'd add a slight shadow to the screenshots. The white background screenshots have a way of disappearing on our website background.
pt-tools-imgs.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Introducing publisher tools: Simple Monetization for content owners and publishers
3 participants