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

Generate a WP_User for the Blog and Application User #709

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

pfefferle
Copy link
Member

WIP!

Proposed changes:

Other information:

  • Have you written new tests for your changes, if applicable?

Testing instructions:

  • Go to '..'

@Menrath
Copy link
Contributor

Menrath commented Mar 18, 2024

I can imagine the improvements of this, as the current implementation took a lot of time to properly understand it. Thanks for working on this. I would ask for one little favour though: if you could find the time to communicate the motivation behind that PR (again in written words) that would be really awesome! :)

@pfefferle
Copy link
Member Author

@Menrath I had/have to rethink this idea, because it won't work on WordPress multi site, because users are global there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants