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

Example for use in Django #38

Merged
merged 1 commit into from
Nov 22, 2023
Merged

Example for use in Django #38

merged 1 commit into from
Nov 22, 2023

Conversation

cinasina
Copy link
Contributor

  • Feature: Add Django example to import 'provinces' and 'cities' data from CSV files.
  • Fix: Rearranging IDs and sorting cities by province.

@sajaddp
Copy link
Owner

sajaddp commented Oct 31, 2023

سلام @cinasina
از اینکه در توسعه این مخزن مشارکت کردین ممنونم.🙏

چند مورد نیاز به اصلاح داره:

  • فایل csv نباید تغییر کند. این موضوع باعث ایجاد مشکل برای افرادی می‌شود که پیش از این از مخزن استفاده کرده‌اند.
  • فایل‌های خالی __init__.py حذف شوند.
  • با توجه به کد، مثال اصلی در واقع کلاس import_data.py هست که زحمت کشیدین نوشتین. لطفا فقط همین کلاس رو به پوشه python منتقل بفرمایید و اسمش رو DjangoCommand.py قرار بدین.

@sajaddp sajaddp changed the title Feature/Fix Example for use in Django Oct 31, 2023
@cinasina
Copy link
Contributor Author

cinasina commented Nov 1, 2023

سلام
وقت بخیر
خواهش میکنم و خوشحالم از این موضوع

در مورد اصلاحات:

  1. از این بابت در یک پوشه جدا و در management و commands قراردادم به این دلیل که به این شکل امکان شناسایی فایل داده توسط جنگو هست و اگر جا به جا کنم، دستور خطا میده که فایلی پیدا نشد. اون فایل های خالی init هم از همین بابت هست و در واقع تنظیمات اصلی خود جنگو هست.

Source

  1. در مورد csv هم در نهایت با نظر شما هست ولی فایل به نظرم مرتب تر و بر اساس چیدمان اولیه باشه، شکل بهتری داره

با احترام

@sajaddp
Copy link
Owner

sajaddp commented Nov 1, 2023

سلام
نکته درستی اشاره کردی.
اما خب این نکته هم باید توجه داشته باشیم که اینجا برای انواع فریم‌ورک‌ها و زبان‌ها داریم مثال قرار می‌دهیم. طبعا کسی که جنگو کار می‌کنه خودش بلد هست که فایل رو کجا منتقل کنه تا شناسایی درست انجام بشه.

در مورد فایل csv چون دوستان از فایل در پروژه‌هاشون استفاده کردن و ممکن هست در تقسیمات کشوری شهری اضافه بشه اگر ما بیایم و تغییر بدیم برای آپدیت کردن براشون دردسر ساز میشه.
اما در ورژن ۲ چون تغییرات سنگین داریم. ان‌شاالله این مشکل حل میشه و افراد طبق نیاز می‌تونن از نسخه ۱ یا ۲ استفاده کنن.

@cinasina
Copy link
Contributor Author

cinasina commented Nov 7, 2023

سلام
وقت بخیر
ممنون از توضیحتون
من تغییرات را مطابق موردی که فرموده بودید انجام دادم.
صرفا فایل django رو به example ها اضافه کردم.

@sajaddp sajaddp merged commit 9c88c35 into sajaddp:main Nov 22, 2023
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.

None yet

2 participants