Merge pull request #3 from amirhossein-moloki/feat/enhance-attribute-model

Feat/enhance attribute model

Author: Yosef-AlSabbah

.env.example

@@ -0,0 +1,90 @@
+# -----------------------------------------------------------------------------
+# D J A N G O
+# -----------------------------------------------------------------------------
+# A secret key for a particular Django installation. This is used to provide cryptographic signing, and should be set to a unique, unpredictable value.
+# Generate a new key here: https://djecrety.ir/
+SECRET_KEY='django-insecure-your-secret-key'
+
+# Controls whether the application is in debug mode. Set to False in production.
+DEBUG=True
+
+# A list of strings representing the host/domain names that this Django site can serve.
+# In production, this should be set to your actual domain name(s).
+# Example for production: ALLOWED_HOSTS=yourdomain.com,www.yourdomain.com
+ALLOWED_HOSTS='*'
+
+
+# -----------------------------------------------------------------------------
+# D A T A B A S E
+# -----------------------------------------------------------------------------
+# URL for connecting to the PostgreSQL database.
+# Format: postgres://<user>:<password>@<host>:<port>/<dbname>
+DATABASE_URL='postgres://user:password@database:5432/db'
+
+
+# -----------------------------------------------------------------------------
+# R E D I S
+# -----------------------------------------------------------------------------
+# URL for connecting to Redis. Used for caching and Celery.
+# Format: redis://<host>:<port>/<db_number>
+REDIS_URL='redis://cache:6379/1'
+
+
+# -----------------------------------------------------------------------------
+# C E L E R Y
+# -----------------------------------------------------------------------------
+# URL for the Celery message broker (uses Redis).
+CELERY_BROKER_URL='redis://cache:6379/0'
+
+# URL for the Celery result backend (uses Redis).
+CELERY_RESULT_BACKEND='redis://cache:6379/0'
+
+
+# -----------------------------------------------------------------------------
+# E M A I L
+# -----------------------------------------------------------------------------
+# URL for configuring email sending.
+# For development, 'consolemail://' prints emails to the console.
+# For production, use a real SMTP server. Example: 'smtp://user:pass@smtp.example.com:587'
+EMAIL_URL='consolemail://'
+
+
+# -----------------------------------------------------------------------------
+# S I T E
+# -----------------------------------------------------------------------------
+# The domain of your site. Used for generating absolute URLs.
+DOMAIN='localhost:8000'
+
+# The name of your site.
+SITE_NAME='Hypex'
+
+
+# -----------------------------------------------------------------------------
+# S O C I A L   A U T H ( G O O G L E )
+# -----------------------------------------------------------------------------
+# Google OAuth2 credentials for social login.
+# Obtain these from the Google API Console: https://console.developers.google.com/
+GOOGLE_OAUTH2_KEY='your-google-oauth2-key'
+GOOGLE_OAUTH2_SECRET='your-google-oauth2-secret'
+
+
+# -----------------------------------------------------------------------------
+# T H I R D - P A R T Y   S E R V I C E S
+# -----------------------------------------------------------------------------
+
+# --- SMS.ir ---
+# Template ID for sending OTP messages via sms.ir.
+SMS_IR_OTP_TEMPLATE_ID='your-sms-ir-otp-template-id'
+
+# --- Postex (Shipping) ---
+# Details for the sender to be used with the Postex shipping service.
+POSTEX_SENDER_NAME='Your Company Name'
+POSTEX_SENDER_PHONE='Your Company Phone'
+POSTEX_SENDER_ADDRESS='Your Company Address'
+POSTEX_SENDER_POSTAL_CODE='Your Company Postal Code'
+POSTEX_FROM_CITY_CODE=1 # City code for the origin of shipments.
+POSTEX_SERVICE_TYPE='standard' # Default shipping service type.
+
+# --- Zibal (Payment Gateway) ---
+# Your merchant ID for the Zibal payment gateway.
+ZIBAL_MERCHANT_ID='your-zibal-merchant-id'

.github/workflows/django.yml

@@ -65,9 +65,6 @@ jobs:
         EMAIL_HOST_USER: ${{ secrets.EMAIL_HOST_USER }}
         EMAIL_HOST_PASSWORD: ${{ secrets.EMAIL_HOST_PASSWORD }}
         DEFAULT_FROM_EMAIL: "Hypex Store <${{ secrets.EMAIL_HOST_USER }}>"
-        STRIPE_PUBLISHABLE_KEY: ${{ secrets.STRIPE_PUBLISHABLE_KEY }}
-        STRIPE_SECRET_KEY: ${{ secrets.STRIPE_SECRET_KEY }}
-        STRIPE_WEBHOOK_SECRET: ${{ secrets.STRIPE_WEBHOOK_SECRET }}
         GOOGLE_OAUTH2_KEY: ${{ secrets.GOOGLE_OAUTH2_KEY }}
         GOOGLE_OAUTH2_SECRET: ${{ secrets.GOOGLE_OAUTH2_SECRET }}
         RABBITMQ_DEFAULT_USER: ${{ secrets.RABBITMQ_DEFAULT_USER }}

.gitignore

@@ -9,3 +9,9 @@ __pycache__/
 *.DS_Store
 *.sock
 .env
+
+# Log files
+*.log
+
+# Database files
+*.rdb

CODE_REVIEW_REPORT.md

@@ -0,0 +1,89 @@
+با احترام،
+
+پس از بررسی دقیق و جامع پروژه شما، مایلم نتایج ارزیابی خود را ارائه دهم. در ابتدا باید بگویم که این پروژه یکی از باکیفیت‌ترین پروژه‌های فروشگاهی است که من بررسی کرده‌ام. معماری آن بسیار تمیز، کدها خوانا و استاندارد، و از تکنولوژی‌های مدرن به بهترین شکل ممکن استفاده شده است. این پروژه یک پایه و اساس فوق‌العاده محکم برای ساخت یک فروشگاه آنلاین حرفه‌ای و مقیاس‌پذیر است.
+
+در ادامه، گزارش کامل ارزیابی به تفکیک بخش‌های درخواستی شما ارائه می‌شود.
+
+---
+
+### **1. نتیجه‌گیری جامع و پیشنهادات**
+
+در این بخش، خلاصه‌ای از یافته‌ها در قالب جداول ارائه شده است.
+
+#### **جدول ۱: قابلیت‌های موجود و وضعیت آن‌ها**
+
+| قابلیت | وضعیت | توضیحات |
+| :--- | :--- | :--- |
+| **مدیریت کاربران (Account)** | کامل | ثبت‌نام با ایمیل/موبایل (OTP)، ورود، خروج، پروفایل، مدیریت آدرس. |
+| **احراز هویت و مجوزها** | کامل | مبتنی بر JWT، مدیریت توکن، مجوزهای دسترسی دقیق و سفارشی. |
+| **مدیریت محصولات (Shop)** | کامل | CRUD محصولات، دسته‌بندی‌ها، تگ‌گذاری، مدیریت موجودی. |
+| **سیستم نقد و بررسی** | کامل | ثبت نظر فقط برای خریداران، امتیازدهی، مدیریت نظرات. |
+| **سبد خرید (Cart)** | کامل | افزودن، حذف، به‌روزرسانی آیتم‌ها (مبتنی بر Session). |
+| **مدیریت کوپن تخفیف** | کامل | اعمال کوپن بر سبد خرید، اعتبارسنجی و مدیریت استفاده. |
+| **فرآیند ثبت سفارش (Orders)** | کامل | ایجاد سفارش از سبد خرید، مدیریت وضعیت، تاریخچه سفارشات. |
+| **سیستم پیشنهاددهنده محصول** | ناقص | زیرساخت با Celery و Redis وجود دارد اما منطق اصلی نیاز به تکمیل دارد. |
+| **جستجو و فیلتر محصولات** | کامل | فیلتر بر اساس قیمت، دسته‌بندی، تگ و موجودی؛ جستجو بر اساس نام. |
+| **پرداخت آنلاین** | کامل | یکپارچه‌سازی با درگاه زیبال، شامل ایجاد پرداخت و تایید سمت سرور. |
+| **ارسال و حمل و نقل** | کامل | یکپارچه‌سازی با Postex برای استعلام هزینه داینامیک و ایجاد خودکار بارنامه. |
+| **چت آنلاین (Real-time)** | ناقص | زیرساخت با Django Channels وجود دارد اما قابلیت‌ها نیاز به توسعه دارند. |
+
+#### **جدول ۲: قابلیت‌های پیشنهادی برای اضافه شدن**
+
+| قابلیت پیشنهادی | اهمیت | توضیحات |
+| :--- | :--- | :--- |
+| **لیست علاقه‌مندی‌ها (Wishlist)** | بالا | به کاربران اجازه می‌دهد محصولات مورد علاقه خود را برای خریدهای آینده ذخیره کنند و نرخ بازگشت کاربر را افزایش می‌دهد. |
+| **جستجوی پیشرفته (Elasticsearch)** | بالا | برای فروشگاه‌های بزرگ، جستجوی متنی سریع و هوشمند (Full-text search) با Elasticsearch تجربه کاربری را متحول می‌کند. |
+| **مدیریت پیشرفته موجودی انبار** | متوسط | قابلیت‌هایی مانند مدیریت چندین انبار، هشدارهای کاهش موجودی و رزرو موجودی هنگام پرداخت. |
+| **سیستم بازاریابی و اطلاع‌رسانی** | متوسط | ارسال ایمیل/پیامک برای کمپین‌های تخفیف، اطلاع‌رسانی موجود شدن محصول و... |
+| **داشبورد گزارشات برای مدیران** | بالا | گزارشات فروش روزانه/ماهانه، محصولات پرفروش، کاربران فعال و تحلیل‌های کلیدی دیگر. |
+
+#### **جدول ۳: نقاط قوت و ضعف پروژه**
+
+| نقاط قوت (Strengths) | نقاط ضعف (Weaknesses) |
+| :--- | :--- |
+| ✅ معماری بسیار تمیز، ماژولار و قابل توسعه | ❌ پوشش تست‌ها ناکافی است (فیلترینگ، موارد مرزی و...). |
+| ✅ کیفیت کد بالا، خوانا و پیروی از استانداردهای PEP 8 | ❌ پیکربندی امنیتی برای محیط پروداکشن صحیح نیست (`DEBUG=True`). |
+| ✅ بهینه‌سازی عملکرد عالی (کشینگ، کوئری‌های بهینه، Celery) | ❌ برخی منطق‌های کسب‌وکار در چند لایه تکرار شده‌اند (مانند بررسی خرید برای نظر). |
+| ✅ سیستم احراز هویت و مجوزهای بسیار امن و مدرن | ❌ منطق محاسبه مالیات به صورت ثابت (Hardcoded) است. |
+| ✅ مستندات API خودکار و بسیار باکیفیت با `drf-spectacular` | |
+| ✅ یکپارچه‌سازی کامل با سرویس‌های پرداخت (Zibal) و حمل و نقل (Postex) | |
+
+---
+
+### **2. امتیازدهی به هر بخش**
+
+| بخش مورد ارزیابی | امتیاز (از ۱ تا ۵) | دلیل امتیاز |
+| :--- | :--- | :--- |
+| **ساختار پروژه و معماری** | ⭐️ 4.5 | معماری MVVM بسیار تمیز است. با افزودن یک لایه سرویس می‌توانست عالی باشد. |
+| **کیفیت کد** | ⭐️ 5 | کدها بسیار خوانا، استاندارد و با رعایت اصول SOLID و DRY نوشته شده‌اند. |
+| **عملکرد و کارایی** | ⭐️ 5 | استفاده هوشمندانه از کش، کوئری‌های بهینه و وظایف پس‌زمینه، عملکرد را تضمین می‌کند. |
+| **امنیت** | ⭐️ 4 | منطق احراز هویت و مجوزها عالی است، اما تنظیمات پروداکشن نیاز به اصلاح فوری دارد. |
+| **تست‌ها و پوشش کد** | ⭐️ 3.5 | تست‌های موجود باکیفیت هستند اما پوشش آن‌ها برای تضمین کامل قابلیت اطمینان، کافی نیست. |
+| **مستندسازی و قابلیت توسعه** | ⭐️ 5 | مستندات API خودکار و عالی است و ساختار پروژه توسعه آینده را بسیار آسان می‌کند. |
+
+---
+
+### **3. راهکارهای عملی برای بهبود**
+
+1.  **اولویت اول: اصلاحات امنیتی**
+    *   **اقدام فوری:** در فایل تنظیمات پروداکشن (`prod.py`)، مقادیر زیر را تنظیم کنید:
+        *   `DEBUG = False`
+        *   `ALLOWED_HOSTS = ['your_domain.com', 'api.your_domain.com']`
+
+2.  **افزایش پوشش تست‌ها**
+    *   تست‌هایی برای **فیلترینگ، جستجو و مرتب‌سازی** در `ProductViewSet` بنویسید.
+    *   تست‌هایی برای **موارد مرزی (Edge Cases)** اضافه کنید (مثلاً ارسال داده نامعتبر به سریالایزرها).
+    *   یک **تست یکپارچه‌سازی (Integration Test)** برای فرآیند کامل "افزودن به سبد -> اعمال کوپن -> ایجاد سفارش" بنویسید.
+
+3.  **بهبود معماری و کاهش تکرار کد**
+    *   یک **لایه سرویس (Service Layer)** به پروژه اضافه کنید. این لایه می‌تواند شامل توابعی باشد که منطق کسب‌وکار را در خود کپسوله می‌کنند. برای مثال:
+        *   یک تابع `can_user_review_product(user, product)` ایجاد کنید و منطق بررسی خرید محصول را در آن قرار دهید تا هم از مدل و هم از ویو فراخوانی شود.
+        *   منطق محاسبه هزینه حمل و نقل و مالیات را از `OrderCreateSerializer` به یک سرویس منتقل کنید تا داینامیک و قابل مدیریت باشد.
+
+4.  **بهبودهای نهایی**
+    *   **محاسبه مالیات:** منطق محاسبه مالیات را از حالت ثابت به یک سیستم داینامیک (مثلاً بر اساس نوع محصول یا استان) تغییر دهید.
+    *   **تکمیل قابلیت چت:** قابلیت چت آنلاین را با افزودن ویژگی‌هایی مانند تاریخچه گفتگو، وضعیت آنلاین بودن کاربران و... تکمیل کنید.
+
+امیدوارم این ارزیابی جامع برای شما مفید بوده باشد. این پروژه پتانسیل بسیار بالایی دارد و با انجام بهبودهای پیشنهادی، به یک محصول در سطح کلاس جهانی تبدیل خواهد شد.
+
+موفق باشید.

README.md

@@ -0,0 +1,44 @@
+# Generated by Django 5.2 on 2025-11-24 18:05
+
+import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("account", "0001_initial"),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name="Address",
+            fields=[
+                (
+                    "id",
+                    models.BigAutoField(
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("province", models.CharField(max_length=100)),
+                ("city", models.CharField(max_length=100)),
+                ("city_code", models.IntegerField(blank=True, null=True)),
+                ("postal_code", models.CharField(max_length=10)),
+                ("full_address", models.TextField()),
+                ("receiver_name", models.CharField(max_length=100)),
+                ("receiver_phone", models.CharField(max_length=15)),
+                (
+                    "user",
+                    models.ForeignKey(
+                        on_delete=django.db.models.deletion.CASCADE,
+                        related_name="addresses",
+                        to=settings.AUTH_USER_MODEL,
+                    ),
+                ),
+            ],
+        ),
+    ]

account/migrations/0002_address.py

@@ -117,3 +117,17 @@ class Meta:
         verbose_name = _('user')
         verbose_name_plural = _('users')
         ordering = ['-pk']
+
+
+class Address(models.Model):
+    user = models.ForeignKey(UserAccount, related_name='addresses', on_delete=models.CASCADE)
+    province = models.CharField(max_length=100)
+    city = models.CharField(max_length=100)
+    city_code = models.IntegerField(null=True, blank=True)
+    postal_code = models.CharField(max_length=10)
+    full_address = models.TextField()
+    receiver_name = models.CharField(max_length=100)
+    receiver_phone = models.CharField(max_length=15)
+
+    def __str__(self):
+        return f'{self.full_address}, {self.city}, {self.province}'

account/models.py

@@ -2,7 +2,7 @@
 from django.db import transaction
 from rest_framework import serializers
 
-from .models import Profile
+from .models import Profile, Address
 
 User = get_user_model()
 
@@ -100,3 +100,9 @@ def update(self, instance, validated_data):
 
 class RefreshTokenSerializer(serializers.Serializer):
     refresh = serializers.CharField(required=True, help_text="Refresh token to be blacklisted")
+
+
+class AddressSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = Address
+        fields = ['id', 'province', 'city', 'city_code', 'postal_code', 'full_address', 'receiver_name', 'receiver_phone']