Skip to content

Add commands for managing signups on multisite #489

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

Merged
swissspidy merged 17 commits into from
May 4, 2024
Merged

Add commands for managing signups on multisite #489

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
f852acf
Add signup command class
ernilambar Mar 15, 2024
660fff5
Update command docs
ernilambar Mar 18, 2024
986adb9
Update success and error messages
ernilambar Mar 19, 2024
5aee1c4
Update readme based on new success messages
ernilambar Mar 19, 2024
e3bfbbc
Update delete method to user callback function
ernilambar Mar 19, 2024
75b5b55
Accept multiple signups in activate command
ernilambar Mar 19, 2024
d5592a6
Update arguments description
ernilambar Mar 19, 2024
a02a3e6
Move signup under user command
ernilambar Apr 8, 2024
32b75d4
Remove Signup fetcher class
ernilambar Apr 9, 2024
ef4f8dc
Update wp-cli version
ernilambar Apr 26, 2024
778ca3e
Revert cli version
ernilambar Apr 26, 2024
adde1d7
Use fetcher from main library
ernilambar Apr 26, 2024
b266645
Use batch results helper for signup activate
ernilambar Apr 26, 2024
0d2e74a
Implement --all in signup delete command
ernilambar Apr 26, 2024
d0d5a84
Use log method instead of success
ernilambar Apr 27, 2024
5fd3657
Fix indenting issue in feature test file
ernilambar Apr 29, 2024
889336b
Introduce --per_page argument in signup list command
ernilambar Apr 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use log method instead of success
  • Loading branch information
@ernilambar
ernilambar committed Apr 27, 2024
commit d0d5a84225e3f44a3ed52e6ced1b7ffc19c09fce
8 changes: 4 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6321,12 +6321,12 @@ wp user signup

# Activate signup.
$ wp user signup activate 2
Success: Signup 2 activated. Password: bZFSGsfzb9xs
Signup 2 activated. Password: bZFSGsfzb9xs
Success: Activated 1 of 1 signups.

# Delete signup.
$ wp user signup delete 3
Success: Signup 3 deleted.
Signup 3 deleted.
Success: Deleted 1 of 1 signups.


Expand All @@ -6350,7 +6350,7 @@ wp user signup activate <signup>...

# Activate signup.
$ wp user signup activate 2
Success: Signup 2 activated. Password: bZFSGsfzb9xs
Signup 2 activated. Password: bZFSGsfzb9xs
Success: Activated 1 of 1 signups.


Expand All @@ -6375,7 +6375,7 @@ wp user signup delete [<signup>...] [--all]

# Delete signup.
$ wp user signup delete 3
Success: Signup 3 deleted.
Signup 3 deleted.
Success: Deleted 1 of 1 signups.


Expand Down
1 change: 0 additions & 1 deletion features/signup.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -189,4 +189,3 @@ Feature: Manage signups in a multisite installation
"""
0
"""

14 changes: 7 additions & 7 deletions src/Signup_Command.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@
*
* # Activate signup.
* $ wp user signup activate 2
* Success: Signup 2 activated. Password: bZFSGsfzb9xs
* Signup 2 activated. Password: bZFSGsfzb9xs
* Success: Activated 1 of 1 signups.
*
* # Delete signup.
* $ wp user signup delete 3
* Success: Signup 3 deleted.
* Signup 3 deleted.
* Success: Deleted 1 of 1 signups.
*
* @package wp-cli
Expand Down Expand Up @@ -211,7 +211,7 @@ public function get( $args, $assoc_args ) {
*
* # Activate signup.
* $ wp user signup activate 2
* Success: Signup 2 activated. Password: bZFSGsfzb9xs
* Signup 2 activated. Password: bZFSGsfzb9xs
* Success: Activated 1 of 1 signups.
*
* @package wp-cli
Expand All @@ -229,7 +229,7 @@ public function activate( $args, $assoc_args ) {
WP_CLI::warning( "Failed activating signup {$signup->signup_id}." );
++$errors;
} else {
WP_CLI::success( "Signup {$signup->signup_id} activated. Password: {$result['password']}" );
WP_CLI::log( "Signup {$signup->signup_id} activated. Password: {$result['password']}" );
++$successes;
}
}
Expand All @@ -252,7 +252,7 @@ public function activate( $args, $assoc_args ) {
*
* # Delete signup.
* $ wp user signup delete 3
* Success: Signup 3 deleted.
* Signup 3 deleted.
* Success: Deleted 1 of 1 signups.
*
* @package wp-cli
Expand Down Expand Up @@ -282,10 +282,10 @@ public function delete( $args, $assoc_args ) {

foreach ( $signups as $signup ) {
if ( $this->delete_signup( $signup ) ) {
WP_CLI::success( "Signup {$signup->signup_id} deleted." );
WP_CLI::log( "Signup {$signup->signup_id} deleted." );
++$successes;
} else {
WP_CLI::error( "Failed deleting signup {$signup->signup_id}." );
WP_CLI::warning( "Failed deleting signup {$signup->signup_id}." );
++$errors;
}
}
Expand Down