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

feat: [CO-675] Let admin set arbitrary PublicService & Virtual hostname #220

Merged
merged 5 commits into from
May 17, 2023

Conversation

aheeva-yuliya
Copy link
Member

@aheeva-yuliya aheeva-yuliya commented May 10, 2023

Now global admin and delegated admin with granted right modifyGlobalConfig could set an arbitrary value for zimbraPublicServiceHostname and zimbraVirtualHostname.

@aheeva-yuliya aheeva-yuliya marked this pull request as ready for review May 12, 2023 09:07
Copy link
Member

@keshavbhatt keshavbhatt left a comment

Choose a reason for hiding this comment

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

lgtm.

@keshavbhatt
Copy link
Member

Please limit the length of the title of this PR. :D

@aheeva-yuliya aheeva-yuliya changed the title feat: [CO-675] allow global admin to set PublicServiceHostname and VirtualHostName not compliantt with domain name feat: [CO-675] allow admins to set arbitrary PublicServiceHostname and VirtualHostName May 12, 2023
@keshavbhatt keshavbhatt changed the title feat: [CO-675] allow admins to set arbitrary PublicServiceHostname and VirtualHostName feat: [CO-675] Let admin set arbitrary PublicService & Virtual hostname May 12, 2023
@aheeva-yuliya aheeva-yuliya force-pushed the feat_CO-675_allow_global_admin_md branch from f80fa5c to 0bd974b Compare May 12, 2023 13:49
@Polpetta
Copy link
Contributor

LGTM after @frisonisland's suggestions

@aheeva-yuliya aheeva-yuliya merged commit 3d0d765 into devel May 17, 2023
@aheeva-yuliya aheeva-yuliya deleted the feat_CO-675_allow_global_admin_md branch May 17, 2023 07:09
@aheeva-yuliya aheeva-yuliya self-assigned this May 17, 2023
@aheeva-yuliya aheeva-yuliya added the enhancement New feature or request label May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants