Skip to content

Conversation

@0-ROK
Copy link
Owner

@0-ROK 0-ROK commented Sep 23, 2025

  • 기존의 window.electronAPI 호출을 getPlatformServices를 통해 통합하여 코드 일관성 향상
  • UpdateNotification 컴포넌트에서 업데이트 이벤트 리스너를 플랫폼 서비스로 변경하여 유지보수성 개선
  • KeyManagerPage, MainPage, ChainContext, HistoryContext, HttpTemplateContext, KeyContext에서 electronAPI 호출을 서비스 메서드로 대체하여 코드 가독성 및 재사용성 향상
  • 새로운 electronServices 모듈 추가로 서비스 관리 및 호출 구조 개선

- 기존의 window.electronAPI 호출을 getPlatformServices를 통해 통합하여 코드 일관성 향상
- UpdateNotification 컴포넌트에서 업데이트 이벤트 리스너를 플랫폼 서비스로 변경하여 유지보수성 개선
- KeyManagerPage, MainPage, ChainContext, HistoryContext, HttpTemplateContext, KeyContext에서 electronAPI 호출을 서비스 메서드로 대체하여 코드 가독성 및 재사용성 향상
- 새로운 electronServices 모듈 추가로 서비스 관리 및 호출 구조 개선
@0-ROK 0-ROK changed the title feat: 플랫폼 서비스 통합 및 업데이트 알림 기능 개선 feat: 일렉트론 화면을 데모용 웹으로 배포 Sep 23, 2025
@0-ROK 0-ROK marked this pull request as draft September 23, 2025 02:02
@0-ROK 0-ROK self-assigned this Sep 23, 2025
- 새로운 browserServices 모듈을 추가하여 브라우저 환경에서의 서비스 관리 및 호출 구조 개선
- getPlatformServices 함수에서 플랫폼 감지를 통해 브라우저 서비스와 전자 서비스 통합
- 메모리 스토리지 및 다양한 서비스 기능을 구현하여 코드 일관성 및 유지보수성 향상
- Vercel을 통한 정적 웹 데모 배포 방법을 문서화하여 브라우저에서 RiSA의 주요 기능을 체험할 수 있도록 지원
- 웹 환경에서 RSA 키 생성 및 암·복호화 기능을 비활성화하고 사용자에게 안내 메시지를 표시하여 혼란을 방지
- webpack 설정을 통해 웹 타겟에 맞는 빌드 및 배포 구조 개선
- UpdateNotification 컴포넌트에서 전자 환경이 아닐 경우 null 반환하여 불필요한 렌더링 방지
- KeyManagerPage 및 MainPage에서 웹 환경에 대한 제한 안내 메시지 추가로 사용자 경험 향상
- webpack 설정에서 publicPath를 웹 타겟에 맞게 '/'에서 './'로 변경하여 빌드 결과물의 경로를 개선
- 이 변경으로 다양한 배포 환경에서의 호환성 향상
- package.json에서 dev:renderer 스크립트를 수정하여 cross-env를 사용해 WEB_TARGET을 웹으로 설정
- 이 변경으로 웹 환경에서의 개발 서버 실행 시 환경 변수를 명확히 지정하여 호환성 향상
- package.json에서 dev:web 스크립트를 추가하여 웹 환경에서의 개발 서버 실행을 개선
- webpack.config.js에서 웹 타겟에 따른 alias 설정을 추가하여 불필요한 모듈을 제외
- App.tsx에서 React.lazy를 사용하여 페이지 및 컴포넌트의 코드 분할을 구현, 로딩 시 스피너 표시로 사용자 경험 향상
- UpdateNotification 컴포넌트를 Suspense로 감싸 불필요한 렌더링 방지
- 각 서비스 생성 함수에서 ensureElectronAPI 호출을 직접 사용하여 코드 간결성 개선
- 불필요한 변수 선언을 제거하고 화살표 함수로 리팩토링하여 가독성 향상
- 전자 API 호출을 서비스 메서드로 통합하여 코드 일관성 유지
@0-ROK 0-ROK marked this pull request as ready for review September 23, 2025 05:57
@0-ROK 0-ROK merged commit 6472d50 into main Sep 23, 2025
4 checks passed
@0-ROK 0-ROK deleted the add-web-demo branch September 23, 2025 06:51
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.

2 participants