Merge pull request #102 from pirogramming/dev

Dev

Author: Jinecastle03

.github/workflows/deploy.yml

@@ -2,7 +2,7 @@ name: CI/CD
 
 on:
   push:
-    branches: [ "feature/pje" ]   # 운영은 main으로 전환 권장
+    branches: [ "main" ]   # 운영은 main으로 전환 권장
   workflow_dispatch:
 
 concurrency:

apps/places/urls.py

@@ -4,9 +4,9 @@
 app_name = 'places'
 
 urlpatterns = [
-    path('', views.main, name='main'),  # 메인 화면
+    path('', views.place_search, name='main'),  # 메인 화면
     path('search/', views.search, name='search'), 
-    path('place-search/', views.place_search, name='place_search'),  # 새로운 장소 검색 화면
+    path('place-search/', views.main, name='place_search'),  # 새로운 장소 검색 화면
     path('<int:pk>/', views.place_detail, name='place_detail'),
     path('<int:pk>/like/', views.toggle_place_like, name='place_like'),
     path('fragment/', views.place_list_fragment, name='place_list_fragment'),

apps/users/adapter.py

@@ -1,44 +1,85 @@
+# apps/users/adapters.py
 from allauth.socialaccount.adapter import DefaultSocialAccountAdapter
-from django.shortcuts import redirect
-from allauth.account.utils import user_username
-from django.utils.text import slugify
-from django.contrib.auth import get_user_model
 from allauth.exceptions import ImmediateHttpResponse
-from django.contrib import messages
 from allauth.socialaccount.models import SocialAccount
+from django.contrib.auth import get_user_model
+from django.shortcuts import redirect
+from django.contrib import messages
+from django.utils.text import slugify
+import uuid
 
 User = get_user_model()
 
 class MySocialAccountAdapter(DefaultSocialAccountAdapter):
     def pre_social_login(self, request, sociallogin):
+        """
+        - 로그인된 상태면 통과
+        - 동일 이메일의 기존 유저가 있는데 현재 provider가 아니면 차단(중복 가입 방지)
+        """
         if request.user.is_authenticated:
-            return redirect('/')
+            return
 
-        email = sociallogin.account.extra_data.get("email")
-        provider = sociallogin.account.provider  # ex: 'google', 'kakao', 'naver'
+        extra = sociallogin.account.extra_data or {}
+        email = extra.get("email") or extra.get("kakao_account", {}).get("email")
+        provider = sociallogin.account.provider  # "kakao" / "google" / "naver" ...
 
-        if email:
-            try:
-                existing_user = User.objects.get(email=email)
+        if not email:
+            # 이메일 없이 들어오면 폼으로 가지 않게 즉시 리다이렉트
+            messages.error(request, "카카오 이메일 제공에 동의해야 로그인할 수 있어요.")
+            raise ImmediateHttpResponse(redirect("/accounts/login/"))
 
-                # ✅ 이 이메일의 소셜 계정이 현재 provider인지 확인
-                if not SocialAccount.objects.filter(user=existing_user, provider=provider).exists():
-                    messages.error(request, "이미 해당 이메일로 가입된 계정이 있습니다. 소셜 로그인할 수 없습니다.")
-                    raise ImmediateHttpResponse(redirect("/accounts/login/"))
+        try:
+            existing = User.objects.get(email__iexact=email)
+            # 같은 이메일의 유저가 있는데, 해당 provider로 연결되지 않았다면 막기
+            if not SocialAccount.objects.filter(user=existing, provider=provider).exists():
+                messages.error(request, "이미 해당 이메일로 가입된 계정이 있어 소셜 로그인할 수 없습니다.")
+                raise ImmediateHttpResponse(redirect("/accounts/login/"))
+        except User.DoesNotExist:
+            pass  # 없으면 계속 진행
 
-            except User.DoesNotExist:
-                pass  # 없으면 자동가입 진행
+    def is_auto_signup_allowed(self, request, sociallogin):
+        """
+        이메일이 있을 때만 추가 폼 없이 자동가입 허용.
+        (없으면 pre_social_login에서 이미 차단됨)
+        """
+        extra = sociallogin.account.extra_data or {}
+        email = extra.get("email") or extra.get("kakao_account", {}).get("email")
+        return bool(email)
 
-    def save_user(self, request, sociallogin, form=None):
-        user = super().save_user(request, sociallogin, form)
+    def populate_user(self, request, sociallogin, data):
+        """
+        소셜 데이터로 User 필수 필드 채우기 (email/username).
+        """
+        user = super().populate_user(request, sociallogin, data)
+
+        extra = sociallogin.account.extra_data or {}
+        kakao_account = extra.get("kakao_account", {})
+        profile = kakao_account.get("profile", {}) or extra.get("properties", {})
+
+        # 이메일 채우기 (필수)
+        email = data.get("email") or kakao_account.get("email")
+        if email and not getattr(user, "email", None):
+            user.email = email
 
-        nickname = sociallogin.account.extra_data.get("name", "")
-        if nickname:
-            user.username = slugify(nickname)
+        # username 자동 생성 (없으면 닉네임/이메일 앞부분/랜덤)
+        if not getattr(user, "username", None):
+            base = (
+                data.get("username")
+                or data.get("name")
+                or (profile.get("nickname") if isinstance(profile, dict) else None)
+                or (email.split("@")[0] if email else "")
+            )
+            user.username = slugify(base or f"user-{uuid.uuid4().hex[:10]}")
+        return user
 
-        if sociallogin.account:
+    def save_user(self, request, sociallogin, form=None):
+        """
+        저장 시 provider / provider_uid도 기록(있을 때만).
+        """
+        user = super().save_user(request, sociallogin, form)
+        if hasattr(user, "provider"):
             user.provider = sociallogin.account.provider
+        if hasattr(user, "provider_uid"):
             user.provider_uid = sociallogin.account.uid
-
         user.save()
         return user

apps/users/forms.py

@@ -3,6 +3,8 @@
 from django.contrib.auth.forms import PasswordResetForm
 from django.contrib.auth import get_user_model
 from django.utils.translation import gettext_lazy as _
+from allauth.account.forms import SignupForm
+from django.utils.text import slugify
 
 User = get_user_model()
 
@@ -22,3 +24,28 @@ def clean_email(self):
             raise forms.ValidationError(_("비밀번호 재설정을 할 수 있는 계정이 아닙니다."))
 
         return email
+
+class EmailSignupForm(SignupForm):
+    """
+    일반 회원가입 화면에서 보여줄 폼.
+    기본적으로 email, password1, password2는 SignupForm에 포함되어 있음.
+    username을 받고 싶으면 required를 켜주면 됨.
+    """
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        # 닉네임/아이디 받고 싶으면 True, 아니라면 False로 두면 숨김
+        if "username" in self.fields:
+            self.fields["username"].required = True
+            self.fields["username"].label = "닉네임"
+            self.fields["username"].help_text = ""
+
+    def save(self, request):
+        user = super().save(request)
+        # 로컬 가입 표시(비번 재설정 허용 기준과 일치)
+        if hasattr(user, "provider") and not user.provider:
+            user.provider = "local"
+        # username unique 보호(필요 시 슬러그 처리)
+        if hasattr(user, "username") and user.username:
+            user.username = slugify(user.username) or user.username
+        user.save()
+        return user

config/settings.py

@@ -104,9 +104,6 @@ def _split_env_list(key):
 ]
 
 WSGI_APPLICATION = 'config.wsgi.application'
-TEMPLATES[0]["OPTIONS"]["context_processors"] += [
-    "apps.places.context_processors.public_settings",
-]
 
 # Database
 # https://docs.djangoproject.com/en/5.2/ref/settings/#databases
@@ -208,12 +205,17 @@ def _split_env_list(key):
 
 
 # 이메일 로그인 관련 설정
-ACCOUNT_LOGIN_METHODS = {"email"}
-ACCOUNT_SIGNUP_FIELDS = ["username*", "email*", "password1*", "password2*"]
+ACCOUNT_AUTHENTICATION_METHOD = "email"
+ACCOUNT_SIGNUP_FIELDS = [] 
 ACCOUNT_UNIQUE_EMAIL = True #이메일 중복 허용 불가
 SOCIALACCOUNT_AUTO_SIGNUP = True
 SOCIALACCOUNT_LOGIN_ON_GET = True 
 SOCIALACCOUNT_ADAPTER = 'apps.users.adapter.MySocialAccountAdapter'
+ACCOUNT_EMAIL_REQUIRED = True
+ACCOUNT_USERNAME_REQUIRED = False         # (소셜 자동가입 시 username 때문에 폼 안 뜨게)
+ACCOUNT_EMAIL_VERIFICATION = "none"       # 개발 중엔 끔(운영 전환 시 "mandatory")
+SOCIALACCOUNT_EMAIL_REQUIRED = True
+SOCIALACCOUNT_QUERY_EMAIL = True
 
 AUTHENTICATION_BACKENDS = (
     #추가 장고에서 사용자의 이름을 기준으로 로그인하도록 설정
@@ -255,7 +257,8 @@ def _split_env_list(key):
 # 제공하는 값이 다르기 때문에 가져올 데이터를 설정한 이후 추가/삭제 해보면 됩니다.
 # SCOPE값에 제공하지 않는 값을 넣거나 하는 이유로 오류가 나올 수 있음
 "SCOPE": [
-
+    "profile_nickname",
+    "account_email",
 ],
 #추가
 "AUTH_PARAMS": {
@@ -330,3 +333,8 @@ def _split_env_list(key):
 CSRF_TRUSTED_ORIGINS = _split_env_list("CSRF_TRUSTED_ORIGINS")
 ACCOUNT_DEFAULT_HTTP_PROTOCOL = "https"
 
+# settings.py
+ACCOUNT_FORMS = {
+    "signup": "apps.users.forms.EmailSignupForm",
+}
+

static/css/main.css

@@ -708,6 +708,9 @@ form[method="get"] > div button[type="submit"]:hover {
   cursor: pointer !important;
   box-shadow: none !important;
   gap: 8px;
+  position: fixed;
+  bottom: 20px;
+  right: 20px;
 }
 
 /* 실제 버튼 모양 */

static/css/style.css

@@ -41,6 +41,9 @@ nav{
 .logo_div { width:150px; height:75px; overflow:hidden;}
 .logo_div img { width:100%; height:100%; object-fit:contain; }
 
+/* 죄측 네비 */
+.left_nav{margin-left: 10px;}
+
 /* 우측 네비 */
 .right_nav{ display:flex; gap:10px; }
 .right_nav a{