Skip to content

Commit

Permalink
fix: site main branch is required
Browse files Browse the repository at this point in the history
  • Loading branch information
Geoffroy Empain committed Dec 10, 2020
1 parent da899fe commit 2420ad4
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions src/entities/sites/site.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,5 @@
import {
alternatives, array, object, string,
} from 'joi';
import {
ARRAY_MAX, COLOR_PATTERN, STRING_MAX_LENGTH, SUBDOMAIN_PATTERN,
} from '../../constants';
import { alternatives, array, object, string } from 'joi';
import { ARRAY_MAX, COLOR_PATTERN, STRING_MAX_LENGTH, SUBDOMAIN_PATTERN } from '../../constants';
import { AppDb } from '../../db/db';
import { Branch } from './branch';

Expand Down Expand Up @@ -75,7 +71,7 @@ export const $siteDomain = object({
export const $site = object({
name: $siteName,
color: string().required().regex(COLOR_PATTERN),
mainBranch: string().required().max(STRING_MAX_LENGTH),
mainBranch: string().optional().max(STRING_MAX_LENGTH),
domains: array().min(0).max(ARRAY_MAX).optional()
.default([])
.items($siteDomain),
Expand Down

0 comments on commit 2420ad4

Please sign in to comment.