-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathREADME.txt
45 lines (24 loc) · 1.66 KB
/
README.txt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
Do at least ONE of the following tasks: refactor is mandatory. Write tests is optional, will be good bonus to see it.
Please do not invest more than 2-4 hours on this.
Upload your results to a Github repo, for easier sharing and reviewing.
Thank you and good luck!
Code to refactor
=================
1) app/Http/Controllers/BookingController.php
2) app/Repository/BookingRepository.php
Code to write tests (optional)
=====================
3) App/Helpers/TeHelper.php method willExpireAt
4) App/Repository/UserRepository.php, method createOrUpdate
----------------------------
What I expect in your repo:
X. A readme with: Your thoughts about the code. What makes it amazing code. Or what makes it ok code. Or what makes it terrible code. How would you have done it. Thoughts on formatting, structure, logic.. The more details that you can provide about the code (what's terrible about it or/and what is good about it) the easier for us to assess your coding style, mentality etc
And
Y. Refactor it if you feel it needs refactoring. The more love you put into it. The easier for us to asses your thoughts, code principles etc
IMPORTANT: Make two commits. First commit with original code. Second with your refactor so we can easily trace changes.
NB: you do not need to set up the code on local and make the web app run. It will not run as its not a complete web app. This is purely to assess you thoughts about code, formatting, logic etc
===== So expected output is a GitHub link with either =====
1. Readme described above (point X above) + refactored code
OR
2. Readme described above (point X above) + refactored core + a unit test of the code that we have sent
Thank you!