Skip to content

Changed the plugin file path for activation and uninstall hooks#378

Merged
Sephsekla merged 4 commits intoWordPress:developfrom
andreg:develop
Jul 26, 2023
Merged

Changed the plugin file path for activation and uninstall hooks#378
Sephsekla merged 4 commits intoWordPress:developfrom
andreg:develop

Conversation

@andreg
Copy link
Copy Markdown
Contributor

@andreg andreg commented Jul 13, 2023

What?

The register_activation_hook and register_uninstall_hook functions now point to the main plugin file, independently of where they are placed in the project.

Why?

This change allows for the database tables to actually be created on plugin activation and destroyed on plugin uninstall.

Copy link
Copy Markdown
Collaborator

@Sephsekla Sephsekla left a comment

Choose a reason for hiding this comment

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

👍

@Sephsekla
Copy link
Copy Markdown
Collaborator

Sorry @andreg, I think the process was unclear on who needed to merge this, and that may have caused some confusion. I'll be updating the contributing documentation to make things clearer going forward.

@andreg
Copy link
Copy Markdown
Contributor Author

andreg commented Jul 25, 2023

To be honest, I didn't even see any button to actually perform the merge.

@Sephsekla Sephsekla merged commit 6553ca6 into WordPress:develop Jul 26, 2023
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.

2 participants