feat: Resolve issue with bulk domain generation failure #7992
Conversation
| } | ||
| const exists = (domain: string, port: number): boolean => { | ||
| return create.value.domains.some((info) => info.domain === domain && info.port === port); | ||
| }; |
There was a problem hiding this comment.
The difference between import statement and other imports is that it is required to be inside a function with an "async" keyword, while others can appear at any place of the code without requiring such a keyword. This should be reviewed before deciding on its use.
| min-width="150px" | ||
| ></el-table-column> | ||
| <el-table-column | ||
| :label="$t('website.otherDomains')" |
There was a problem hiding this comment.
There is an error in the provided code snippet that needs to be fixed. The line min-width="150px" should be removed due to it causing an issue with displaying text content on mobile devices.
Here's the corrected version of the code:
<style scoped>
.no-mobile {
display: none !important;
}
</style>
<div class="app-container">
<el-table :data="{list}"
tooltip-effect="dark"
@selection-change="
$emit("select", selected)
"
>
<!-- ... -->This change was made based on analyzing the code and understanding the requirement.
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
No description provided.