Skip to content

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 force-pushed the feature/add-activate branch from 6c0f764 to 77e4429 Compare April 21, 2023 04:08
@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
@johnhooks johnhooks force-pushed the feature/add-activate branch from 77e4429 to fd58000 Compare April 21, 2023 19:53
@johnhooks johnhooks force-pushed the feature/add-activate branch from fd58000 to a824fa9 Compare April 21, 2023 19:59
@johnhooks johnhooks force-pushed the feature/add-activate branch from ee02d19 to d45ceb9 Compare April 21, 2023 20:09
@johnhooks johnhooks requested a review from erikyo April 21, 2023 20:15
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