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.

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

Jump to bottom

[Bug] v-add-domain doesn't fail if mail / dns domain allready exists #3667

Closed
jaapmarcus opened this issue Jun 10, 2023 · 1 comment · Fixed by #3670
Closed

[Bug] v-add-domain doesn't fail if mail / dns domain allready exists #3667

jaapmarcus opened this issue Jun 10, 2023 · 1 comment · Fixed by #3670
Labels
bug Something isn't working

Comments

@jaapmarcus
Copy link
Member

Describe the bug

x

Tell us how to replicate the bug

x

Which components are affected by this bug?

Control Panel Backend, Control Panel Command Line Interface

Hestia Control Panel Version

1.7.7

Operating system

N/A

Log capture

No response
@jaapmarcus jaapmarcus added the bug Something isn't working label Jun 10, 2023
@jaapmarcus
Copy link
Member Author

Bug was caused due a bug in v-import-cpanel

jaapmarcus added a commit to jaapmarcus/hestiacp that referenced this issue Jun 10, 2023
@jaapmarcus
Fix: hestiacp#3667 and hestiacp#3666
d1bf6ea 
- Check if user contains an email in case it is optimal
- Check if v-add-web-domain runs successfully if not exit
@jaapmarcus jaapmarcus linked a pull request Jun 10, 2023 that will close this issue
Fix: #3667 and #3666 #3670
Merged
ScIT-Raphael pushed a commit that referenced this issue Jun 11, 2023
@jaapmarcus
Fix: #3667 and #3666 (#3670)
d26cc7e 
- Check if user contains an email in case it is optimal
- Check if v-add-web-domain runs successfully if not exit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: #3667 and #3666 jaapmarcus/hestiacp
1 participant
@jaapmarcus