Skip to content
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

feature(mvp): Stage 4 - add activator and uninstaller #298

Merged
merged 7 commits into from
Apr 21, 2023

Conversation

johnhooks
Copy link
Collaborator

@johnhooks johnhooks commented Apr 21, 2023

What?

Adds plugin activator and uninstaller. The database SQL to create the database tables for the plugin are in the Database\Schema static class.

Why?

Need to be able to install the plugin and have a database to contain the notification data.

Reopening of #199

@johnhooks johnhooks changed the title Feature/add activate feature(mvp): Stage 2 - add activator and uninstaller Apr 21, 2023
@johnhooks johnhooks changed the title feature(mvp): Stage 2 - add activator and uninstaller feature(mvp): Stage 4 - add activator and uninstaller Apr 21, 2023
@johnhooks johnhooks mentioned this pull request Apr 21, 2023
38 tasks
@johnhooks johnhooks added the [Scope] Database / Persistence Issue related to storing data. To WP database or another location. label Apr 21, 2023
fix: uninstall drop table names

fix: format error

chore: remove logging

fix: add uninstall hook

fix: use of deactivate in uninstall

fix: remove logging function

feature: add support for multisite

fix: Use Yoda Condition checks, you must

fix: add/remove options in each multisite instance

chore: fix comment formating

fix: nullable notification message meta

test: add activate and uninstall tests

refactor: add type hint to db-test-case

fix: package name PHPDoc comments
Copy link
Collaborator

@erikyo erikyo left a comment

Choose a reason for hiding this comment

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

LGTM!

@johnhooks johnhooks merged commit fedf494 into WordPress:develop Apr 21, 2023
@johnhooks johnhooks deleted the feature/add-activate branch April 21, 2023 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Scope] Database / Persistence Issue related to storing data. To WP database or another location.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants