Skip to content

Comments

Use campaign name instead of creator's display_name for unlock button#170

Open
beberry wants to merge 9 commits intomasterfrom
jstikans-use-campaign-name
Open

Use campaign name instead of creator's display_name for unlock button#170
beberry wants to merge 9 commits intomasterfrom
jstikans-use-campaign-name

Conversation

@beberry
Copy link
Member

@beberry beberry commented May 8, 2025

Problem

The plugin uses user.display_name in the unlock flow.

Solution

Use campaign.name instead.

Before
Screenshot 2025-05-08 at 10 54 41 AM

After
Screenshot 2025-05-08 at 10 33 10 AM

@beberry beberry force-pushed the jstikans-remove-unused-string branch from 75e4762 to 5a72a97 Compare May 8, 2025 04:49
@beberry beberry force-pushed the jstikans-use-campaign-name branch from 2747f94 to 196cec2 Compare May 8, 2025 04:54
@beberry beberry force-pushed the jstikans-remove-unused-string branch from 5a72a97 to 0818a29 Compare May 8, 2025 05:45
@beberry beberry force-pushed the jstikans-use-campaign-name branch from 196cec2 to 11a8ba3 Compare May 8, 2025 05:46
@beberry beberry marked this pull request as ready for review May 8, 2025 17:55
// This function checks and saves creator's full name, name and surname. These are used in post locking interface

if (!get_option('patreon-creator-full-name', false) or '' == get_option('patreon-creator-full-name', false)) {
if (!get_option('patreon-creator-name', false) or !get_option('patreon-creator-full-name', false) or '' == get_option('patreon-creator-full-name', false)) {
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
if (!get_option('patreon-creator-name', false) or !get_option('patreon-creator-full-name', false) or '' == get_option('patreon-creator-full-name', false)) {
if (!get_option('patreon-campaign-name', false) or '' == get_option('patreon-campaign-name', false)) {

@beberry beberry force-pushed the jstikans-remove-unused-string branch from 0818a29 to 58c0f53 Compare May 8, 2025 21:26
Base automatically changed from jstikans-remove-unused-string to master May 8, 2025 21:27
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.

3 participants