-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
profile added #29
profile added #29
Conversation
سلام عباس جان، دمت گرم. در مورد پروفایل کاربر، فکر می کنم به جز بخش لیدر برد با اسم و عکس کاربر کاری نداشته باشیم. من قصد نداشتم ذخیره کنم این دیتارو. اما خیلی خوبه. در مورد اون متنهایی هم که گفتی من قبلا فکرش رو کرده بودم اینجا هم توضیح دادم : #12 چک می کنم کد رو اگه سوالی داشتم همینجا صحبت می کنیم . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
خیلی عالی. فقط کاش داخل
outputHelper
از
EOT
استفاده نمی کردی. من اصلاحش می کنم.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😍ایول اصلا حواسم به match نبود
@@ -20,6 +23,10 @@ final class User extends Model | |||
|
|||
public function __construct() | |||
{ | |||
if ($this->name == 'unknown') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
نفهمیدم چرا اینجا unknown بودن رو چک کردی ؟
یوزر های فعلی name شون null مگه نیست ؟ اینجوری باید داخل دیتابیس یه دیفالت در نظر می گرفتی برای name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
یوزای جدید موقه درست شدن اسم پروفایلشون گذاشته میشه .
مشکل یوزر هایی که الان توی ربات هستن . اینا رو یا باید با یه اسکریپت جدا به همشون مقدار داد از طریق اسم تلگرام یا مقدار پیشفرض درنظر گرفته براش . من راه حل دومی رو انتخاب کردم منتهی یادم رفت مقدار پیشفرض بزارم
توی مرج بعدی درستش میکنم
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
عباس جان درستش کردم دمت گرم
public function change(): void | ||
{ | ||
$table = $this->table('users'); | ||
$table->addColumn('name', 'string') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
اینجا یه پارامتر default='unknown' نیازه گمونم
❤️❤️ |
تغییرات انجام شده :
دیتا سیدر ها با توجه به تغییرات میگریشن فیکس شدن
InputHelper
ریفکتور شدبخش پروفایل کاربر با توجه به #24 ایجاد شد
متد های دریافت عکس پروفایل و ارسال عکس به
TelegramHelper
اضافه شددرحال حاظر اسم کاربر از
first_name
تلگرام گرفته میشه همچنین عکس پروفایل تلگرام کاربر به عنوان عکس پروفایل در ربات درنظر گرفته میشهقدم های بعدی و مشکلات پیشرو :
کاربر باید بتونه عکس و اسم پروفایل خودش رو تغییر بده. این قابلیت با کارهایی که تا الان انجام شده یکم فرق داره . تا الان هر متن دلخواه ( هر متنی که توی منو نباشه و تعریف نشده باشه ) به عنوان جواب سوال درنظر گرفته میشد. از الان باید تفکیک بشه یعنی باید چک بشه که این متنی که کاربر ارسال کرده جواب بازی هست یا اسمیه که انتخاب کرده.
بعدا هم ممکنه بخش های دیگه هم بخایم اضافه کنیم که نیاز به پاسخ کاربر داشته باشه ( مثلا پیام به ما از داخل ربات یا پیشنهاد مرحله ) پس بهتره یک فکر کلی برای این مورد بکنیم.