Skip to content

Add commands to handle the trusted servers from command line - #40796

Merged
pako81 merged 5 commits into
masterfrom
trusted_servers_cli_handling
Jun 5, 2023
Merged

Add commands to handle the trusted servers from command line#40796
pako81 merged 5 commits into
masterfrom
trusted_servers_cli_handling

Conversation

@jvillafanez

@jvillafanez jvillafanez commented May 17, 2023

Copy link
Copy Markdown
Member

Description

New commands to handle trusted servers from the command line

Related Issue

https://github.com/owncloud/enterprise/issues/5731

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE
@jvillafanez
jvillafanez force-pushed the trusted_servers_cli_handling branch from 4ecdb3d to 21f675c Compare May 17, 2023 15:29
@jvillafanez
jvillafanez marked this pull request as ready for review May 17, 2023 16:15
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from ownclouders May 18, 2023
@owncloud owncloud deleted a comment from update-docs Bot May 18, 2023
Comment thread apps/federation/lib/Command/TrustedServerRemove.php Outdated
Comment thread apps/federation/lib/Command/TrustedServerRemove.php Outdated
@phil-davis
phil-davis self-requested a review May 18, 2023 08:06

@phil-davis phil-davis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM - I suppose that others should also review this.

Comment thread apps/federation/lib/Command/TrustedServerRemove.php
@pako81

pako81 commented May 18, 2023

Copy link
Copy Markdown

Apart from a small remark for the remove command, it looks ok to me 👍

@sonarqubecloud

Copy link
Copy Markdown
@mrow4a

mrow4a commented May 26, 2023

Copy link
Copy Markdown
Contributor
@pako81

pako81 commented May 26, 2023

Copy link
Copy Markdown

merge this?

@pako81
pako81 merged commit 1624c63 into master Jun 5, 2023
@delete-merged-branch
delete-merged-branch Bot deleted the trusted_servers_cli_handling branch June 5, 2023 08:58
shokri-navid pushed a commit to pondersource/core that referenced this pull request Jul 16, 2023
…d#40796)

* Add commands to handle the trusted servers from command line

* Add changelog entry

* Add unit tests

* Improve messages

* Add additional message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants