Add save version dialog

2024-11-22 15:10:43 +08:00
12 changed files with 163 additions and 280 deletions
--- a/backend/app/api/api_v1/routers/auth.py
+++ b/backend/app/api/api_v1/routers/auth.py
@@ -25,15 +25,11 @@ async def login(
        minutes=security.ACCESS_TOKEN_EXPIRE_MINUTES
    )
    if user.is_superuser:
-        roles = "super"
-        permissions = "ALL"
+        permissions = "admin"
    else:
-        roles = ";".join(role.name for role in user.roles)
-        perlst = [perm.privilege for role in user.roles for perm in role.permissions]
-        permissions =";".join(list(set(perlst)))
-    
+        permissions = "user"
    access_token = security.create_access_token(
-        data={"sub": user.id, "roles":roles,"permissions": permissions ,},
+        data={"sub": user.id, "permissions": permissions},
        expires_delta=access_token_expires,
    )

--- a/backend/app/api/api_v1/routers/platform.py
+++ b/backend/app/api/api_v1/routers/platform.py
@@ -23,7 +23,7 @@ platform_router = r = APIRouter()
 )
 async def apps_list(
    request: Request,
-    user = Depends(get_current_active_user),
+    user = Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -60,7 +60,7 @@ async def apps_list(
 async def apps_update(
    request: Request,
    app: AppVersion,
-    user=Depends(get_current_active_user),
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -68,21 +68,7 @@ async def apps_update(
    except Exception as e:
        raise APIException('platform:apps',request.url._url,f"Error occurred while get create app :",e)
    
-@r.delete(
-    "/apps/{domainurl}/{appid}", response_model_exclude_none=True
-)
-async def apps_delete(
-    request: Request,
-    domainurl:str,
-    appid: str,
-    user=Depends(get_current_active_user),
-    db=Depends(get_db),
-):
-    try:
-        return delete_apps(db, domainurl,appid)
-    except Exception as e:
-        raise APIException('platform:apps',request.url._url,f"Error occurred while delete apps({domainurl}:{appid}):",e)
-        
+    
@r.get(
    "/appsettings/{id}",
    response_model=App,
@@ -197,7 +183,7 @@ async def flow_details(
 async def flow_list(
    request: Request,
    appid: str,
-    user=Depends(get_current_active_user),
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -212,8 +198,8 @@ async def flow_list(
@r.post("/flow", response_model=Flow, response_model_exclude_none=True)
 async def flow_create(
    request: Request,
-    flow: FlowIn,
-    user=Depends(get_current_active_user),
+    flow: FlowBase,
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -229,13 +215,13 @@ async def flow_create(
 async def flow_edit(
    request: Request,
    flowid: str, 
-    flow: FlowIn,
-    user=Depends(get_current_active_user),
+    flow: FlowBase,
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
        domain = get_activedomain(db, user.id)
-        return edit_flow(db,domain.url, flow,user.id)
+        return edit_flow(db,domain.url, flow)
    except Exception as e:
        raise APIException('platform:flow',request.url._url,f"Error occurred while edit flow:",e)

@@ -273,7 +259,7 @@ async def domain_details(
 async def domain_create(
    request: Request,
    domain: DomainBase,
-    user=Depends(get_current_active_user),
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -288,11 +274,10 @@ async def domain_create(
 async def domain_edit(
    request: Request,
    domain: DomainBase,
-    user=Depends(get_current_active_user),
    db=Depends(get_db),
 ):
    try:
-        return edit_domain(db, domain,user.id)
+        return edit_domain(db, domain)
    except Exception as e:
        raise APIException('platform:domain',request.url._url,f"Error occurred while edit domain:",e)

@@ -318,7 +303,7 @@ async def domain_delete(
 async def userdomain_details(
    request: Request,
    userId: Optional[int] = Query(None, alias="userId"),
-    user=Depends(get_current_active_user),
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -366,7 +351,7 @@ async def userdomain_delete(
 async def get_useractivedomain(
    request: Request,
    userId: Optional[int] = Query(None, alias="userId"),
-    user=Depends(get_current_active_user),
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
@@ -386,7 +371,7 @@ async def update_activeuserdomain(
    request: Request,
    domainid:int,
    userId: Optional[int] = Query(None, alias="userId"),
-    user=Depends(get_current_active_user),
+    user=Depends(get_current_user),
    db=Depends(get_db),
 ):
    try:
--- a/backend/app/api/api_v1/routers/users.py
+++ b/backend/app/api/api_v1/routers/users.py
@@ -1,4 +1,4 @@
-from fastapi import APIRouter, Request, Depends, Response, Security, encoders
+from fastapi import APIRouter, Request, Depends, Response, encoders
 import typing as t

 from app.db.session import get_db
@@ -8,11 +8,9 @@ from app.db.crud import (
    create_user,
    delete_user,
    edit_user,
-    assign_userrole,
-    get_roles,
 )
-from app.db.schemas import UserCreate, UserEdit, User, UserOut,Role
-from app.core.auth import get_current_user,get_current_active_user, get_current_active_superuser
+from app.db.schemas import UserCreate, UserEdit, User, UserOut
+from app.core.auth import get_current_active_user, get_current_active_superuser

 users_router = r = APIRouter()

@@ -25,14 +23,14 @@ users_router = r = APIRouter()
 async def users_list(
    response: Response,
    db=Depends(get_db),
-    current_user=Depends(get_current_active_user),
+    current_user=Depends(get_current_active_superuser),
 ):
    """
    Get all users
    """
-    users = get_users(db,current_user.is_superuser)
+    users = get_users(db)
    # This is necessary for react-admin to work
-    #response.headers["Content-Range"] = f"0-9/{len(users)}"
+    response.headers["Content-Range"] = f"0-9/{len(users)}"
    return users


@@ -107,30 +105,3 @@ async def user_delete(
    Delete existing user
    """
    return delete_user(db, user_id)
-
-
-@r.post("/userrole",
-    response_model=User,
-    response_model_exclude_none=True,)
-async def assign_role(
-    request: Request,
-    userid:int,
-    roles:t.List[int],
-    db=Depends(get_db)
-):
-   
-    return assign_userrole(db,userid,roles)
-
-
-@r.get(
-    "/roles",
-    response_model=t.List[Role],
-    response_model_exclude_none=True,
-)
-async def roles_list(
-    response: Response,
-    db=Depends(get_db),
-    current_user=Security(get_current_active_user, scopes=["role_list"]),
-):
-    roles = get_roles(db)
-    return roles
--- a/backend/app/core/auth.py
+++ b/backend/app/core/auth.py
@@ -1,6 +1,5 @@
-from fastapi.security import SecurityScopes
 import jwt
-from fastapi import Depends, HTTPException, Request, Security, status
+from fastapi import Depends, HTTPException, status
 from jwt import PyJWTError

 from app.db import models, schemas, session
@@ -8,7 +7,7 @@ from app.db.crud import get_user_by_email, create_user,get_user
 from app.core import security


-async def get_current_user(security_scopes: SecurityScopes,
+async def get_current_user(
    db=Depends(session.get_db), token: str = Depends(security.oauth2_scheme)
 ):
    credentials_exception = HTTPException(
@@ -17,21 +16,13 @@ async def get_current_user(security_scopes: SecurityScopes,
        headers={"WWW-Authenticate": "Bearer"},
    )
    try:
-
        payload = jwt.decode(
            token, security.SECRET_KEY, algorithms=[security.ALGORITHM]
        )
        id: int = payload.get("sub")
        if id is None:
            raise credentials_exception
-
        permissions: str = payload.get("permissions")
-        if not permissions =="ALL":
-            for scope in security_scopes.scopes:
-                if scope not in permissions.split(";"):
-                    raise HTTPException(
-                        status_code=403, detail="The user doesn't have enough privileges"
-                    )
        token_data = schemas.TokenData(id = id, permissions=permissions)
    except PyJWTError:
        raise credentials_exception
--- a/backend/app/db/crud.py
+++ b/backend/app/db/crud.py
@@ -1,4 +1,3 @@
-import datetime
 from fastapi import HTTPException, status
 from sqlalchemy.orm import Session
 from sqlalchemy import and_
@@ -20,12 +19,9 @@ def get_user_by_email(db: Session, email: str) -> schemas.UserBase:


 def get_users(
-    db: Session, super:bool
+    db: Session, skip: int = 0, limit: int = 100
 ) -> t.List[schemas.UserOut]:
-    if super:
-        return db.query(models.User).all()
-    else:
-        return db.query(models.User).filter(models.User.is_superuser == False)
+    return db.query(models.User).offset(skip).limit(limit).all()


 def create_user(db: Session, user: schemas.UserCreate):
@@ -73,48 +69,28 @@ def edit_user(
    db.refresh(db_user)
    return db_user

-
-def get_roles(
-    db: Session
-) -> t.List[schemas.Role]:
-    return db.query(models.Role).all()
-
-def assign_userrole( db: Session, user_id: int, roles: t.List[int]):
-    db_user = db.query(models.User).get(user_id)
-    if db_user:
-        for role in db_user.roles:
-            db_user.roles.remove(role)
-        for roleid in roles:
-            role = db.query(models.Role).get(roleid)
-            if role:
-                db_user.roles.append(role)
-    db.commit()
-    db.refresh(db_user)
-    return db_user
-
 def get_apps(
    db: Session,
-    domainurl:str
+    domain_url:str
 ) -> t.List[schemas.AppList]:
-    return db.query(models.App).filter(models.App.domainurl == domainurl).all()
+    return db.query(models.App).filter(models.App.domainurl == domain_url).all()

 def update_appversion(db: Session, appedit: schemas.AppVersion,userid:int):
-    db_app = db.query(models.App).filter(and_(models.App.domainurl ==  appedit.domainurl,models.App.appid == appedit.appid)).first()
-    if not db_app:
-       raise HTTPException(status.HTTP_404_NOT_FOUND, detail="User not found")
-    
-    db_app.version = db_app.version + 1
-    appversion = models.AppVersion(
+    app = db.query(models.App).filter(and_(models.App.domainurl ==  appedit.domainurl,models.App.appid == appedit.appid)).first()
+    if app:
+        app.version = app.version + 1
+        db_app = app
+        appver = app.version
+    else:
+        appver = 1
+        db_app = models.App(
        domainurl = appedit.domainurl,
        appid=appedit.appid,
-        appname=db_app.appname,
-        version = db_app.version,
-        versionname = appedit.versionname,
-        comment = appedit.comment,
-        updateuserid = userid,
-        createuserid = userid
-    )
-    db.add(appversion)
+        appname=appedit.appname,
+        version = 1,
+        updateuser= userid
+        )
+
    db.add(db_app)

    flows = db.query(models.Flow).filter(and_(models.Flow.domainurl ==  appedit.domainurl,models.App.appid == appedit.appid)) 
@@ -127,9 +103,7 @@ def update_appversion(db: Session, appedit: schemas.AppVersion,userid:int):
        name = flow.name,
        content = flow.content,
        createuser = userid,
-        version = db_app.version, 
-        updateuserid = userid,
-        createuserid = userid
+        version = appver
        )
        db.add(db_flowhistory)

@@ -137,17 +111,6 @@ def update_appversion(db: Session, appedit: schemas.AppVersion,userid:int):
    db.refresh(db_app)
    return db_app

-def delete_apps(db: Session, domainurl: str,appid: str ):
-    db_app = db.query(models.App).filter(and_(models.App.domainurl == domainurl,models.App.appid ==appid)).first()
-    if not db_app:
-        raise HTTPException(status.HTTP_404_NOT_FOUND, detail="App not found")
-    db.delete(db_app)
-    db_flows = db.query(models.Flow).filter(and_(models.Flow.domainurl == domainurl,models.Flow.appid ==appid))
-    for flow in db_flows:
-        db.delete(flow)
-    db.commit()
-    return db_app
-
 def get_appsetting(db: Session, id: int):
    app = db.query(models.AppSetting).get(id)
    if not app:
@@ -203,28 +166,16 @@ def get_actions(db: Session):
    return actions


-def create_flow(db: Session, domainurl: str, flow: schemas.FlowIn,userid:int):
+def create_flow(db: Session, domainurl: str, flow: schemas.FlowBase):
    db_flow = models.Flow(
        flowid=flow.flowid,
        appid=flow.appid,
        eventid=flow.eventid,
        domainurl=domainurl,
        name=flow.name,
-        content=flow.content,
-        createuserid = userid,
-        updateuserid = userid
+        content=flow.content
    )
    db.add(db_flow)
-    db_app = db.query(models.App).filter(and_(models.App.domainurl ==  domainurl,models.App.appid == flow.appid)).first()
-    if not db_app:
-        db_app = models.App(
-        domainurl = domainurl,
-        appid=flow.appid,
-        appname=flow.appname,
-        version = 0,
-        createuserid= userid,
-        updateuserid = userid
-        )
    db.commit()
    db.refresh(db_flow)
    return db_flow
@@ -239,20 +190,17 @@ def delete_flow(db: Session, flowid: str):


 def edit_flow(
-    db: Session, domainurl: str, flow: schemas.FlowIn,userid:int
+    db: Session, domainurl: str, flow: schemas.FlowBase
 ) -> schemas.Flow:
    db_flow = get_flow(db, flow.flowid)
    if not db_flow:
        #見つからない時新規作成
-        return create_flow(db,domainurl,flow,userid)
+        return create_flow(db,domainurl,flow)
    
-    db_flow.appid =flow.appid,
-    db_flow.eventid=flow.eventid,
-    db_flow.domainurl=domainurl,
-    db_flow.name=flow.name,
-    db_flow.content=flow.content,
-    db_flow.updateuserid = userid,
-    db_flow.update_time = datetime.now
+    update_data = flow.dict(exclude_unset=True)
+
+    for key, value in update_data.items():
+        setattr(db_flow, key, value)

    db.add(db_flow)
    db.commit()
@@ -285,9 +233,7 @@ def create_domain(db: Session, domain: schemas.DomainBase,userid:int):
        name=domain.name,
        url=domain.url,
        kintoneuser=domain.kintoneuser,
-        kintonepwd=domain.kintonepwd,
-        createuserid = userid,
-        updateuserid = userid
+        kintonepwd=domain.kintonepwd
    )
    db.add(db_domain)
    db.flush()
@@ -306,19 +252,18 @@ def delete_domain(db: Session,id: int):


 def edit_domain(
-    db: Session, domain: schemas.DomainBase,userid:int
+    db: Session, domain: schemas.DomainBase
 ) -> schemas.Domain:
    domain.encrypt_kintonepwd()
    db_domain = db.query(models.Domain).get(domain.id)
    if not db_domain:
-       raise HTTPException(status.HTTP_404_NOT_FOUND, detail="Domain not found")
-    db_domain.tenantid = domain.tenantid
-    db_domain.name=domain.name
-    db_domain.url=domain.url
-    db_domain.kintoneuser=domain.kintoneuser
-    db_domain.kintonepwd = domain.kintonepwd
-    db_domain.updateuserid = userid
-    db_domain.update_time = datetime.now
+        raise HTTPException(status.HTTP_404_NOT_FOUND, detail="Domain not found")
+    update_data = domain.dict(exclude_unset=True)
+
+    for key, value in update_data.items():
+        if key != "id" and not (key == "kintonepwd" and (value is None or value == "")):
+            setattr(db_domain, key, value)
+    print(str(db_domain))
    db.add(db_domain)
    db.commit()
    db.refresh(db_domain)
--- a/backend/app/db/models.py
+++ b/backend/app/db/models.py
@@ -1,8 +1,8 @@
-from sqlalchemy import Boolean, Column, Integer, String, DateTime,ForeignKey,Table
+from sqlalchemy import Boolean, Column, Integer, String, DateTime,ForeignKey
 from sqlalchemy.ext.declarative import as_declarative
 from sqlalchemy.orm import relationship
 from datetime import datetime
-from app.db.session import Base
+
 from app.core.security import chacha20Decrypt

@as_declarative()
@@ -11,21 +11,6 @@ class Base:
    create_time = Column(DateTime, default=datetime.now)
    update_time = Column(DateTime, default=datetime.now, onupdate=datetime.now)

-
-userrole = Table(
-    "userrole",
-    Base.metadata,
-    Column("userid",Integer,ForeignKey("user.id")),
-    Column("roleid",Integer,ForeignKey("role.id")),
-)
-
-rolepermission = Table(
-    "rolepermission",
-    Base.metadata,
-    Column("roleid",Integer,ForeignKey("role.id")),
-    Column("permissionid",Integer,ForeignKey("permission.id")),
-)
-
 class User(Base):
    __tablename__ = "user"

@@ -35,25 +20,6 @@ class User(Base):
    hashed_password = Column(String(200), nullable=False)
    is_active = Column(Boolean, default=True)
    is_superuser = Column(Boolean, default=False)
-    roles = relationship("Role",secondary=userrole,back_populates="users")
-
-
-class Role(Base):
-    __tablename__ = "role"
-
-    name = Column(String(100))
-    description = Column(String(255))
-    users = relationship("User",secondary=userrole,back_populates="roles")
-    permissions = relationship("Permission",secondary=rolepermission,back_populates="roles")
-
-class Permission(Base):
-    __tablename__ = "permission"
-
-    menu = Column(String(100))
-    function = Column(String(255))
-    privilege = Column(String(100))
-    roles = relationship("Role",secondary=rolepermission,back_populates="permissions")
-

 class App(Base):
    __tablename__ = "app"
@@ -62,25 +28,8 @@ class App(Base):
    appname = Column(String(200), nullable=False)
    appid = Column(String(100), index=True, nullable=False)
    version = Column(Integer)
-    createuserid = Column(Integer,ForeignKey("user.id"))
-    updateuserid = Column(Integer,ForeignKey("user.id"))
-    createuser = relationship('User',foreign_keys=[createuserid]) 
-    updateuser = relationship('User',foreign_keys=[updateuserid]) 
-
-class AppVersion(Base):
-    __tablename__ = "appversion"
-
-    domainurl = Column(String(200), nullable=False) 
-    appname = Column(String(200), nullable=False)
-    appid = Column(String(100), index=True, nullable=False)
-    version = Column(Integer)
-    versionname = Column(String(200), nullable=False)
-    comment = Column(String(200), nullable=False)
-    createuserid = Column(Integer,ForeignKey("user.id"))
-    updateuserid = Column(Integer,ForeignKey("user.id"))
-    createuser = relationship('User',foreign_keys=[createuserid]) 
-    updateuser = relationship('User',foreign_keys=[updateuserid]) 
-
+    updateuser = Column(Integer,ForeignKey("user.id"))
+    user = relationship('User')

 class AppSetting(Base):
    __tablename__ = "appsetting"
@@ -115,11 +64,7 @@ class Flow(Base):
    eventid = Column(String(100), index=True, nullable=False)
    domainurl = Column(String(200)) 
    name = Column(String(200))   
-    content = Column(String) 
-    createuserid = Column(Integer,ForeignKey("user.id"))
-    updateuserid = Column(Integer,ForeignKey("user.id"))
-    createuser = relationship('User',foreign_keys=[createuserid]) 
-    updateuser = relationship('User',foreign_keys=[updateuserid]) 
+    content = Column(String)  

 class FlowHistory(Base):
    __tablename__ = "flowhistory"
@@ -130,11 +75,8 @@ class FlowHistory(Base):
    domainurl = Column(String(200)) 
    name = Column(String(200))   
    content = Column(String)  
+    createuser = Column(Integer,ForeignKey("user.id"))
    version = Column(Integer)
-    createuserid = Column(Integer,ForeignKey("user.id"))
-    updateuserid = Column(Integer,ForeignKey("user.id"))
-    createuser = relationship('User',foreign_keys=[createuserid]) 
-    updateuser = relationship('User',foreign_keys=[updateuserid]) 

 class Tenant(Base):
    __tablename__ = "tenant"
@@ -156,10 +98,7 @@ class Domain(Base):
    def decrypt_kintonepwd(self):
        decrypted_pwd = chacha20Decrypt(self.kintonepwd)
        return decrypted_pwd
-    createuserid = Column(Integer,ForeignKey("user.id"))
-    updateuserid = Column(Integer,ForeignKey("user.id"))
-    createuser = relationship('User',foreign_keys=[createuserid]) 
-    updateuser = relationship('User',foreign_keys=[updateuserid]) 
+

 class UserDomain(Base):
    __tablename__ = "userdomain"
--- a/backend/app/db/schemas.py
+++ b/backend/app/db/schemas.py
@@ -8,26 +8,13 @@ class Base(BaseModel):
    create_time: datetime
    update_time: datetime

-
-class Permission(BaseModel):
-    id: int
-    menu:str
-    function:str
-    privilege:str
-
-class Role(BaseModel):
-    id: int
-    name:str
-    description:str
-    permissions:t.List[Permission] = []
-    
 class UserBase(BaseModel):
    email: str
    is_active: bool = True
    is_superuser: bool = False
    first_name: str = None
    last_name: str = None
-    roles:t.List[Role] = []
+

 class UserOut(UserBase):
    pass
@@ -67,16 +54,13 @@ class AppList(Base):
    domainurl: str
    appname: str
    appid:str
-    updateuser: UserOut
    version:int
-    
+    user:UserOut
+
 class AppVersion(BaseModel):
    domainurl: str
    appname: str
-    versionname: str
-    comment:str
    appid:str
- 

 class TokenData(BaseModel):
    id:int = 0
@@ -122,11 +106,9 @@ class Action(BaseModel):
    class ConfigDict:
        orm_mode = True

-class FlowIn(BaseModel):
+class FlowBase(BaseModel):
    flowid: str 
-    # domainurl:str
    appid: str 
-    appname:str
    eventid: str
    name: str = None
    content: str = None
--- a/frontend/src/components/dialog/VersionInput.vue
+++ b/frontend/src/components/dialog/VersionInput.vue
@@ -0,0 +1,39 @@
+<template>
+  <q-input
+    v-model="versionInfo.name"
+    filled
+    label="バージョン名"
+    :rules="[(val) => val.length <= 20 || '20字以内で入力ください']"
+  />
+  <q-input
+    v-model="versionInfo.desc"
+    filled
+    type="textarea"
+    :rules="[(val) => val.length <= 80 || '80字以内で入力ください']"
+    label="説明"
+  />
+</template>
+<script setup lang="ts">
+import { ref, watch, defineProps, defineEmits } from 'vue';
+import { QInput } from 'quasar';
+import { IVersionInfo } from 'src/types/AppTypes';
+
+const props = defineProps<{
+  modelValue: IVersionInfo;
+}>();
+
+const versionInfo = ref({
+  ...props.modelValue,
+  name: props.modelValue.name || `新バージョン ${new Date().toLocaleString()}`,
+ });
+
+const emit = defineEmits(['update:modelValue']);
+
+watch(
+  versionInfo,
+  () => {
+    emit('update:modelValue', { ...versionInfo.value });
+  },
+  { immediate: true, deep: true }
+);
+</script>
--- a/frontend/src/pages/AppManagement.vue
+++ b/frontend/src/pages/AppManagement.vue
@@ -83,7 +83,7 @@ const getApps = async () => {
      id: item.appid,
      name: item.appname,
      url: `${item.domainurl}/k/${item.appid}`,
-      user: `${item.updateuser.first_name} ${item.updateuser.last_name}` ,
+      user: `${item.user.first_name} ${item.user.last_name}` ,
      updatetime:date.formatDate(item.update_time, 'YYYY/MM/DD HH:mm'),
      version: Number(item.version)
    }
--- a/frontend/src/pages/FlowChart.vue
+++ b/frontend/src/pages/FlowChart.vue
@@ -14,6 +14,15 @@
            <q-space></q-space>
            <q-btn-dropdown color="primary" label="保存" icon="save" :loading="saveLoading" >
              <q-list>
+                <q-item clickable v-close-popup @click="onSaveVersion">
+                  <q-item-section avatar >
+                    <q-icon name="history"></q-icon>
+                  </q-item-section>
+                  <q-item-section>
+                    <q-item-label>新バージョン保存</q-item-label>
+                  </q-item-section>
+                </q-item>
+
                <q-item clickable v-close-popup @click="onSaveFlow">
                  <q-item-section avatar >
                    <q-icon name="save" color="primary"></q-icon>
@@ -75,6 +84,10 @@
      </template>
      <action-select ref="appDg" name="model" :filter="filter"  type="single" @clearFilter="onClearFilter" ></action-select>
    </ShowDialog>
+    <!-- save version dialog -->
+    <ShowDialog v-model:visible="saveVersionAction" name="新バージョン保存" @close="closeSaveVersionDg" min-width="500px">
+      <version-input v-model="versionInfo" />
+    </ShowDialog>
    <q-inner-loading
      :showing="initLoading"
      color="primary"
@@ -87,7 +100,7 @@
 import { ref, reactive, computed, onMounted } from 'vue';
 import { useRoute } from 'vue-router';
 import { IActionNode, ActionNode, IActionFlow, ActionFlow, RootAction, IActionProperty } from 'src/types/ActionTypes';
-import { IManagedApp } from 'src/types/AppTypes';
+import { IManagedApp, IVersionInfo } from 'src/types/AppTypes';
 import { storeToRefs } from 'pinia';
 import { useFlowEditorStore } from 'stores/flowEditor';
 import { useAuthStore } from 'stores/useAuthStore';
@@ -98,6 +111,7 @@ import ShowDialog from 'components/ShowDialog.vue';
 import ActionSelect from 'components/ActionSelect.vue';
 import PropertyPanel from 'components/right/PropertyPanel.vue';
 import EventTree from 'components/left/EventTree.vue';
+import VersionInput from 'components/dialog/VersionInput.vue';
 import { FlowCtrl } from '../control/flowctrl';
 import { useQuasar } from 'quasar';

@@ -105,6 +119,7 @@ const deployLoading = ref(false);
 const saveLoading = ref(false);
 const initLoading = ref(true);
 const drawerLeft = ref(false);
+const versionInfo = ref<IVersionInfo>();
 const $q = useQuasar();
 const store = useFlowEditorStore();
 const authStore = useAuthStore();
@@ -117,6 +132,7 @@ const prevNodeIfo = ref({
 });
 // const refFlow = ref<ActionFlow|null>(null);
 const showAddAction = ref(false);
+const saveVersionAction = ref(false);
 const drawerRight = ref(false);
 const filter=ref("");
 const model = ref("");
@@ -177,7 +193,7 @@ const onDeleteAllNextNodes = (node: IActionNode) => {
 }
 const closeDg = (val: any) => {
  console.log("Dialog closed->", val);
-  if (val == 'OK') {
+  if (val == 'OK' && appDg?.value?.selected?.length > 0) {
    const data = appDg.value.selected[0];
    const actionProps = JSON.parse(data.property);
    const outputPoint = JSON.parse(data.outputPoints);
@@ -245,6 +261,20 @@ const onSaveActionProps=(props:IActionProperty[])=>{
  }
 };

+const onSaveVersion = async () => {
+  versionInfo.value = {
+    id: '1' // TODO
+  }
+  saveVersionAction.value = true;
+  // await onSaveAllFlow();
+}
+
+const closeSaveVersionDg = (val: 'OK'|'CANCEL') => {
+  if (val == 'OK') {
+    console.log(versionInfo.value);
+  }
+}
+
 const onSaveFlow = async () => {
  const targetFlow = store.selectedFlow;
  if (targetFlow === undefined) {
--- a/frontend/src/stores/flowEditor.ts
+++ b/frontend/src/stores/flowEditor.ts
@@ -128,7 +128,6 @@ export const useFlowEditorStore = defineStore('flowEditor', {
      const jsonData = {
        flowid: isNew ? flow.createNewId() : flow.id,
        appid: this.appInfo?.appId,
-        appname: this.appInfo?.name,
        eventid: root?.name,
        name: root?.subTitle,
        content: JSON.stringify(flow),
--- a/frontend/src/types/AppTypes.ts
+++ b/frontend/src/types/AppTypes.ts
@@ -1,14 +1,20 @@
-interface IUser {
-  first_name: string;
-  last_name: string;
-  email: string;
-}
-
-export interface IManagedApp {
-  appid: string;
-  appname: string;
-  domainurl: string;
-  version: string;
-  updateuser: IUser;
-  update_time: string;
-}
+interface IUser {
+  first_name: string;
+  last_name: string;
+  email: string;
+}
+
+export interface IManagedApp {
+  appid: string;
+  appname: string;
+  domainurl: string;
+  version: string;
+  user: IUser;
+  update_time: string;
+}
+
+export interface IVersionInfo {
+  id: string;
+  name?: string;
+  desc?: string;
+}