Skip to content

Added paybutton prefix to generic option names #56

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.

Sign up for GitHub

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

Jump to bottom
Merged
Klakurka merged 1 commit into from
Apr 20, 2025
Merged

Added paybutton prefix to generic option names #56

Klakurka merged 1 commit into from
Apr 20, 2025

Conversation

@xecdev
Copy link
Collaborator

@xecdev xecdev commented Apr 20, 2025 •
edited
Loading

This PR fixes #54 by renaming exisitng option names to include the paybutton_ prefix. Every stored option now begins with the unique paybutton_ prefix and old installs are migrated silently (back‑compatible).

Test plan:

  • Remove/deactivate the plugin
  • Install the updated plugin
  • Go to Paywall Settings
  • Make sure that the Admin wallet address and Background Color and Text Color for the 'Unlocked Content Indicator' (ofc if you have it enabled) are migrated.

@xecdev xecdev added the enhancement (behind the scenes) Stuff that users won't see label Apr 20, 2025
@xecdev xecdev requested a review from Klakurka April 20, 2025 07:47
@xecdev xecdev self-assigned this Apr 20, 2025
@Klakurka Klakurka merged commit 2fba8fb into master Apr 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Klakurka Klakurka Klakurka approved these changes

Assignees

@xecdev xecdev

Labels

enhancement (behind the scenes) Stuff that users won't see

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Generic function/class/define/namespace/option names

3 participants

@xecdev @Klakurka