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

profile added #29

Merged
merged 6 commits into from
Apr 9, 2022
Merged

profile added #29

merged 6 commits into from
Apr 9, 2022

Conversation

abbasudo
Copy link
Collaborator

@abbasudo abbasudo commented Apr 8, 2022

تغییرات انجام شده :
دیتا سیدر ها با توجه به تغییرات میگریشن فیکس شدن
InputHelper ریفکتور شد
بخش پروفایل کاربر با توجه به #24 ایجاد شد
متد های دریافت عکس پروفایل و ارسال عکس به TelegramHelper اضافه شد

درحال حاظر اسم کاربر از first_name تلگرام گرفته میشه همچنین عکس پروفایل تلگرام کاربر به عنوان عکس پروفایل در ربات درنظر گرفته میشه
قدم های بعدی و مشکلات پیشرو :
کاربر باید بتونه عکس و اسم پروفایل خودش رو تغییر بده. این قابلیت با کارهایی که تا الان انجام شده یکم فرق داره . تا الان هر متن دلخواه ( هر متنی که توی منو نباشه و تعریف نشده باشه ) به عنوان جواب سوال درنظر گرفته میشد. از الان باید تفکیک بشه یعنی باید چک بشه که این متنی که کاربر ارسال کرده جواب بازی هست یا اسمیه که انتخاب کرده.
بعدا هم ممکنه بخش های دیگه هم بخایم اضافه کنیم که نیاز به پاسخ کاربر داشته باشه ( مثلا پیام به ما از داخل ربات یا پیشنهاد مرحله ) پس بهتره یک فکر کلی برای این مورد بکنیم.

@abbasudo abbasudo changed the title minor bug fix profile added Apr 8, 2022
@geeksesi
Copy link
Owner

geeksesi commented Apr 9, 2022

سلام عباس جان، دمت گرم.

در مورد پروفایل کاربر، فکر می کنم به جز بخش لیدر برد با اسم و عکس کاربر کاری نداشته باشیم. من قصد نداشتم ذخیره کنم این دیتارو. اما خیلی خوبه.

در مورد اون متنهایی هم که گفتی من قبلا فکرش رو کرده بودم اینجا هم توضیح دادم : #12

چک می کنم کد رو اگه سوالی داشتم همینجا صحبت می کنیم .

Copy link
Owner

@geeksesi geeksesi left a comment

Choose a reason for hiding this comment

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

خیلی عالی. فقط کاش داخل
outputHelper
از
EOT
استفاده نمی کردی. من اصلاحش می کنم.

Copy link
Owner

@geeksesi geeksesi left a 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') {
Copy link
Owner

Choose a reason for hiding this comment

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

نفهمیدم چرا اینجا unknown بودن رو چک کردی ؟

یوزر های فعلی name شون null مگه نیست ؟ اینجوری باید داخل دیتابیس یه دیفالت در نظر می گرفتی برای name

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

یوزای جدید موقه درست شدن اسم پروفایلشون گذاشته میشه .
مشکل یوزر هایی که الان توی ربات هستن . اینا رو یا باید با یه اسکریپت جدا به همشون مقدار داد از طریق اسم تلگرام یا مقدار پیشفرض درنظر گرفته براش . من راه حل دومی رو انتخاب کردم منتهی یادم رفت مقدار پیشفرض بزارم
توی مرج بعدی درستش میکنم

Copy link
Owner

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')
Copy link
Owner

Choose a reason for hiding this comment

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

اینجا یه پارامتر default='unknown' نیازه گمونم

@geeksesi geeksesi merged commit b14ffc9 into geeksesi:master Apr 9, 2022
geeksesi added a commit that referenced this pull request Apr 9, 2022
@abbasudo
Copy link
Collaborator Author

abbasudo commented Apr 9, 2022

❤️❤️
توی لیدربرد اگه بخایم به ازای هر نفر متد getUser از تلگرام صدا زده بشه ربات میپوکه
برای همین به نظرم رسید یه جا ذخیره بشه
قابلیت عوض کردن هم داشته باشه عالی میشه

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.

2 participants