From 497161dade119ddba255ec18ec1a1d3d22d7a017 Mon Sep 17 00:00:00 2001
From: steven <stevenlgtm@gmail.com>
Date: Tue, 4 Oct 2022 20:10:29 +0800
Subject: [PATCH] chore: limit workspace name length

---
 server/workspace.go                              | 4 ++++
 web/src/components/Header.tsx                    | 6 +++---
 web/src/components/UpsertWorkspaceUserDialog.tsx | 2 +-
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/server/workspace.go b/server/workspace.go
index 4471266..c577787 100644
--- a/server/workspace.go
+++ b/server/workspace.go
@@ -26,6 +26,10 @@ func (s *Server) registerWorkspaceRoutes(g *echo.Group) {
 		if err := json.NewDecoder(c.Request().Body).Decode(workspaceCreate); err != nil {
 			return echo.NewHTTPError(http.StatusBadRequest, "Malformatted post workspace request").SetInternal(err)
 		}
+		if len(workspaceCreate.Name) > 20 {
+			return echo.NewHTTPError(http.StatusBadRequest, "Workspace name length should be less than 20")
+		}
+
 		workspace, err := s.Store.CreateWorkspace(ctx, workspaceCreate)
 		if err != nil {
 			return echo.NewHTTPError(http.StatusInternalServerError, "Failed to create workspace").SetInternal(err)
diff --git a/web/src/components/Header.tsx b/web/src/components/Header.tsx
index c1d4283..4036c1e 100644
--- a/web/src/components/Header.tsx
+++ b/web/src/components/Header.tsx
@@ -36,8 +36,8 @@ const Header: React.FC = () => {
     <>
       <div className="w-full bg-amber-50">
         <div className="w-full max-w-4xl mx-auto px-3 py-5 flex flex-row justify-between items-center">
-          <div className="flex flex-row justify-start items-center">
-            <Link to={"/"} className="text-base font-mono font-medium cursor-pointer">
+          <div className="flex flex-row justify-start items-center shrink mr-2">
+            <Link to="/" className="text-base font-mono font-medium cursor-pointer">
               Corgi
             </Link>
             {workspaceList.length > 0 && activedWorkspace !== undefined && (
@@ -78,7 +78,7 @@ const Header: React.FC = () => {
               </>
             )}
           </div>
-          <div className="relative">
+          <div className="relative flex-shrink-0">
             {user ? (
               <Dropdown
                 trigger={
diff --git a/web/src/components/UpsertWorkspaceUserDialog.tsx b/web/src/components/UpsertWorkspaceUserDialog.tsx
index 870e437..811579a 100644
--- a/web/src/components/UpsertWorkspaceUserDialog.tsx
+++ b/web/src/components/UpsertWorkspaceUserDialog.tsx
@@ -88,7 +88,7 @@ const UpsertWorkspaceUserDialog: React.FC<Props> = (props: Props) => {
           <input
             className="w-full rounded border text-sm shadow-inner px-2 py-2"
             type="number"
-            value={state.workspaceUserUpsert.userId}
+            value={state.workspaceUserUpsert.userId <= 0 ? "" : state.workspaceUserUpsert.userId}
             onChange={handleUserIdInputChange}
           />
         </div>