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

가입 승인제 PR 보강 #28

Merged
merged 5 commits into from
Jan 1, 2025
Merged

가입 승인제 PR 보강 #28

merged 5 commits into from
Jan 1, 2025

Conversation

seojangho
Copy link
Collaborator

@seojangho seojangho commented Nov 29, 2024

Resolves #27

@seojangho
Copy link
Collaborator Author

번역 추가했으나 https://pr-28.test.nekoplanet.xyz/assets/locales/en-US.2024.11.0+neko-rc.json 에는 여전히 일본어가 보이네..?

@seojangho seojangho changed the base branch from update/24.11.0 to neko January 1, 2025 06:06
- yaml config 에서 approvalRequiredForSignup 제거
  - config을 사용하고 있던 부분을 제거하고, DB의 설정값을 대신 사용
- MkVisitorDashboard 에서 approvalRequiredForSignup 에 따라 컨트롤이
	가입 버튼이 다르게 보이도록
- 제어판에서 approvalRequiredForSignup 을 켜면 켜지도록
@seojangho seojangho force-pushed the update/24.11.0-fix-approval branch from 73bbb80 to 743440a Compare January 1, 2025 06:07
@seojangho seojangho marked this pull request as ready for review January 1, 2025 08:30
Copy link

@HotoRas HotoRas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HotoRas HotoRas merged commit 653155e into neko Jan 1, 2025
25 of 26 checks passed
@seojangho seojangho deleted the update/24.11.0-fix-approval branch January 1, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

가입 승인제 관련 보강사항
2 participants