Friends App
task name
The code is submitted in a dedicated feature branch.
Only code files are submitted.
Please, review.
Hi! Feedback received, bugs fixed. Review please. Self check done!
Hey!
Congratulations on your PR! 😎😎😎
Let's do some self-checks to fix most common issues and to make some improvements to the code before reviewers put their hands on the code.
Go through the requirements/most common mistakes linked below and fix the code as appropriate.
If you have any questions to requirements/common mistakes feel free asking them here or in Students' chat.
When you genuinely believe you are done put a comment stating that you have completed self-checks and fixed code accordingly.
Also, be aware, that if you would silently ignore this recommendation, a mentor can think that you are still working on fixes. And your PR will not be reviewed. 😒
Please, make sure you haven't made common mistakes.
In addition to the bot's suggestions above, please post a link to the app demo in Students' chat, collect feedback and fix bugs if any based on that. When user tests are done post here a comment on that.
Universal recommendations:
- Make sure your code follows General Requirements
- Use common sense or seek for an advice whenever requirements look ambiguous or unclear.
- Pay more attention to code style - descriptive variable names, indentations, empty spaces, etc. Code should look good :)
By the way, you may proceed to the next task before this one is reviewed and merged.
Sincerely yours, Submissions Kottachecker 😺
I have completed self-checks and fixed code accordingly.
This issue has been automatically marked as stale because there were no activity during last 14 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.
А. Чому так? Найбільш розповсюджена причина: Студент не реагує на коментарі змінами коду і не задає запитань через брак часу або зміну життєвих пріоритетів. Покинуті піари відволікають менторів. Коли у студента з'явиться час, він/вона зможе перевідкрити той самий піар і продовжити роботу.
Б. Що робити, якщо в піарі нема оновлень, оскільки не зрозуміло, що треба зробити? Варіант 1. Задати питання в самому PR. Варіант 2. Задати питання в студентському чаті.
В. А якщо я все зробив(ла) і це ментор не рев'юває мої зміни?
- Переконайся, що ти відреагував(ла) на всі коментарі або кодом, або запитаннями, або відповідями. Напиши в PR і в чаті, що чесно вважаєш, що все зроблено і попроси повторне рев'ю. Якщо щось не зрозуміло, задай запитання.
- Реагуй на коментарі як менторів, так і інших учасників, включаючи ботів.
- Не ігноруй прохання типу * "Let's do some self-checks ..." * "Go through the checklist below..." * "mark fulfilled requirements..." * "if you would silently ignore this recommendation, a mentor may think that you are still working on fixes" навіть якщо вони написані ботом. Боти помічники і ментори покладаються на те, що прохання і пропозиції бота дотримуються. Не лінись піти по лінках в коментарях, погуглити термінологію та скористатись Google Translate.
- Можливо, у менторів склалися інші пріоритети через роботу, сімейні обставини і т.п. В такому разі, якщо ти зробив(ла) рекомендоване вище, то волай в чаті, що PR позначений stale, наче, все зроблено, а ментори чомусь не реагують - рятуйте!
Г. Хіба недостатньо того, що я додав(ла) коміт із змінами? Часто буває так, що бачиш новий коміт, ідеш перевіряти, змін багато, доводиться перечитувати весь код. А потім з'ясовується, що одна невеличка зміна "відкладена на потім" чи з'являється ще один коміт і знов треба перечитувати все. Любіть нас, спілкуйтеся з нами - і ми відповімо повною взаємністю.
Традиційна пропозиція: задай питання по вищенаписаному в студентському чаті.
@OleksiyRudenko review pls
This issue has been automatically marked as stale because there were no activity during last 14 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.
А. Чому так? Найбільш розповсюджена причина: Студент не реагує на коментарі змінами коду і не задає запитань через брак часу або зміну життєвих пріоритетів. Покинуті піари відволікають менторів. Коли у студента з'явиться час, він/вона зможе перевідкрити той самий піар і продовжити роботу.
Б. Що робити, якщо в піарі нема оновлень, оскільки не зрозуміло, що треба зробити? Варіант 1. Задати питання в самому PR. Варіант 2. Задати питання в студентському чаті.
В. А якщо я все зробив(ла) і це ментор не рев'юває мої зміни?
- Переконайся, що ти відреагував(ла) на всі коментарі або кодом, або запитаннями, або відповідями. Напиши в PR і в чаті, що чесно вважаєш, що все зроблено і попроси повторне рев'ю. Якщо щось не зрозуміло, задай запитання.
- Реагуй на коментарі як менторів, так і інших учасників, включаючи ботів.
- Не ігноруй прохання типу * "Let's do some self-checks ..." * "Go through the checklist below..." * "mark fulfilled requirements..." * "if you would silently ignore this recommendation, a mentor may think that you are still working on fixes" навіть якщо вони написані ботом. Боти помічники і ментори покладаються на те, що прохання і пропозиції бота дотримуються. Не лінись піти по лінках в коментарях, погуглити термінологію та скористатись Google Translate.
- Можливо, у менторів склалися інші пріоритети через роботу, сімейні обставини і т.п. В такому разі, якщо ти зробив(ла) рекомендоване вище, то волай в чаті, що PR позначений stale, наче, все зроблено, а ментори чомусь не реагують - рятуйте!
Г. Хіба недостатньо того, що я додав(ла) коміт із змінами? Часто буває так, що бачиш новий коміт, ідеш перевіряти, змін багато, доводиться перечитувати весь код. А потім з'ясовується, що одна невеличка зміна "відкладена на потім" чи з'являється ще один коміт і знов треба перечитувати все. Любіть нас, спілкуйтеся з нами - і ми відповімо повною взаємністю.
Традиційна пропозиція: задай питання по вищенаписаному в студентському чаті.