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.

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

fix: 그누보드6 설치 DB_TABLE_PREFIX 미적용 오류 수정 #569

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

KimTom89
Copy link
Collaborator

@KimTom89 KimTom89 commented Jun 4, 2024

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요. PR을 보내기 전에 모든 항목을 확인해야 합니다.

  • 동일한 업데이트/변경에 대한 다른 Pull Requests가 열려있는지 확인했습니다.
  • 테스트가 성공적으로 수행되었는지 확인했습니다.

PR 유형

어떤 유형의 PR인가요? (해당 항목에 모두 체크해주세요)

  • 버그 수정
  • 새로운 기능
  • UI/UX 개선
  • 문서내용 수정
  • 코드 의미에 영향을 주지 않는 변경사항 (오타, 서식 지정, 변수명 변경 등)
  • 코드 리팩토링 (버그 수정이나 기능 변경 없는 코드 변경)
  • 빌드 관련 변경
  • 테스트 코드 추가
  • 기타 (이유를 설명해주세요.)

변경 사항

  • 설치 시 이미 선언/할당된 메타데이터의 테이블 이름을 갱신하도록 코드 추가
# 테이블 이름을 변경 & 메타데이터 갱신
tables = Base.metadata.tables.values()
for table in tables:
    new_table_name = table.name.replace("g6_", form_data.db_table_prefix)
    table.name = new_table_name

Base.metadata.create_all(bind=engine)
  • core/models.py, install/router.py 코드스타일 일부 수정

관련 이슈

기타 정보

KimTom89 added 2 commits May 31, 2024 18:37
- Base Metadata의 Table name/fullname 속성을 갱신
- core/models.py 코드스타일 일부 수정
@KimTom89 KimTom89 merged commit e5da238 into master Jun 4, 2024
@KimTom89 KimTom89 deleted the fix/PY-44_table_prefix branch June 4, 2024 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant