Skip to content

Conversation

@bexsoft
Copy link
Collaborator

@bexsoft bexsoft commented Feb 15, 2022

What does this do?

Added delete all replication rules capability

How does it look?

2022-02-15 13-28-29 2022-02-15 13_29_21
2022-02-15 12-57-43 2022-02-15 12_58_56

Signed-off-by: Benjamin Perez benjamin@bexsoft.net

@bexsoft bexsoft self-assigned this Feb 15, 2022
@bexsoft bexsoft added the WIP This PR is WIP and cannot be merged yet label Feb 15, 2022
@bexsoft bexsoft changed the title Added delete all replication rules capability [WIP] Added delete all replication rules capability Feb 15, 2022
@harshavardhana
Copy link
Member

We should perhaps ask for more confirmation @bexsoft - asking them to type "yes" or "delete"

@bexsoft bexsoft changed the title [WIP] Added delete all replication rules capability Added delete all replication rules capability Feb 15, 2022
@bexsoft bexsoft added WIP This PR is WIP and cannot be merged yet and removed WIP This PR is WIP and cannot be merged yet labels Feb 15, 2022
@bexsoft bexsoft changed the title Added delete all replication rules capability [WIP] Added delete all replication rules capability Feb 15, 2022
@bexsoft
Copy link
Collaborator Author

bexsoft commented Feb 15, 2022

We should perhaps ask for more confirmation @bexsoft - asking them to type "yes" or "delete"

Fixed @harshavardhana
Screen Shot 2022-02-15 at 14 14 56
Screen Shot 2022-02-15 at 14 14 48

@bexsoft bexsoft changed the title [WIP] Added delete all replication rules capability Added delete all replication rules capability Feb 15, 2022
@bexsoft bexsoft removed the WIP This PR is WIP and cannot be merged yet label Feb 15, 2022
@bexsoft bexsoft added the WIP This PR is WIP and cannot be merged yet label Feb 15, 2022
Signed-off-by: Benjamin Perez <benjamin@bexsoft.net>
@bexsoft bexsoft removed the WIP This PR is WIP and cannot be merged yet label Feb 16, 2022
@dvaldivia dvaldivia merged commit 226e8ee into minio:master Feb 16, 2022
@bexsoft bexsoft deleted the delete-last-rule branch April 5, 2022 19:01
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.

4 participants