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.

Sign up for GitHub

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

Jump to bottom

🚧 Add long time job #23

Merged
merged 42 commits into from
Feb 3, 2020
Merged

🚧 Add long time job #23

merged 42 commits into from
Feb 3, 2020

Conversation

AminovE99
Copy link
Contributor

@AminovE99 AminovE99 commented Jan 13, 2020 •
edited by madmath03
Loading

Signed-off-by: Emil amionv99@gmail.com

Please provide enough information so that others can review your pull request:

Explain the details for making this change. What existing problem does the pull request solve?

Screenshots/GIFs

AminovE99 and others added 20 commits January 13, 2020 14:48
@AminovE99
🚧 Add long time job
2d1a1fa 
Signed-off-by: Emil <amionv99@gmail.com>
@madmath03
🚧 Add TDD functions
4a47e49 
Signed-off-by: mathieu.brunot <mathieu.brunot@monogramm.io>
@AminovE99
🚧 Add test for bg job
3deb8dd 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Add Todo for future
41a488f 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 fix unused variable
035feda 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Add another test
8ef702d 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Fix test
e3b0895 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Fix tests
50ce944 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Trying fix test
9f62767 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Trying fix test v2
bc764fe 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Trying fix test v3
34a9651 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Remove time library
08a8151 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Blank commit
5c3a9c9 
Signed-off-by: Emil <amionv99@gmail.com>
@madmath03
Merge branch 'develop' into feature/job_to_bg_worker
6f91d2f 
Signed-off-by: mathieu.brunot <mathieu.brunot@monogramm.io>
@madmath03
Merge branch 'feature/job_to_bg_worker' of github.com:Monogramm/ERPNe…
4726677 
…xt-OCR into feature/job_to_bg_worker

Signed-off-by: mathieu.brunot <mathieu.brunot@monogramm.io>
@AminovE99
💄 Update tests for validation
5343d66 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Add new tests
ef2cad1 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Update tests
4918b9b 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Last update test
0f222e2 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
💄 The last beautify of code
eb72741 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99 AminovE99 marked this pull request as ready for review January 24, 2020 14:09
@AminovE99 AminovE99 requested a review from madmath03 January 24, 2020 14:09
madmath03
madmath03 requested changes Jan 24, 2020
View reviewed changes
Copy link
Member

@madmath03 madmath03 left a comment

Choose a reason for hiding this comment

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

@AminovE99

erpnext_ocr/erpnext_ocr/doctype/ocr_read/test_ocr_read.py Outdated Show resolved Hide resolved
erpnext_ocr/erpnext_ocr/doctype/ocr_read/test_ocr_read.py Outdated Show resolved Hide resolved
erpnext_ocr/erpnext_ocr/doctype/ocr_read/test_ocr_read.py Outdated Show resolved Hide resolved
erpnext_ocr/erpnext_ocr/doctype/ocr_read/test_ocr_read.py Outdated Show resolved Hide resolved
erpnext_ocr/erpnext_ocr/doctype/ocr_read/test_ocr_read.py Outdated Show resolved Hide resolved
AminovE99 and others added 7 commits January 24, 2020 17:48
@AminovE99
✅ Update tests
50742ed 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Remove unnecesary library
fcaa6df 
Signed-off-by: Emil <amionv99@gmail.com>
@madmath03
👌 Fix unit tests process for bg jobs
e6128c6
@AminovE99
✅ Update tests
d8bbc0d 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
:consturction: Add logs for debuging
94fd70b 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🐳 Add more time for tests
bad4f0a 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🐳 Add bench doctor
5e39492 
Signed-off-by: Emil <amionv99@gmail.com>
AminovE99 and others added 15 commits January 27, 2020 17:12
@AminovE99
🔧 Enable schedulers
5a49a0b 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🔧 Remove exec function from travis yml
a4163be 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🔧 Add bench doctor
75f5350 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🔧 Add bench doctor before enable scheduler
b326b04 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Add tests for OCR Read
333c430 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Mark every tests by print
a1af764 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Remove tests
9ad6064 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Add getter instead of variable
3b12e65 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 Logging tests and requeueing jobs
a17948f 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
🚧 print values of worker
630b2e5 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
✅ Remove while construction and add now parameter
5278ed3 
Signed-off-by: Emil <amionv99@gmail.com>
@AminovE99
💄 Remove old libraries
db2bc2d 
Signed-off-by: Emil <amionv99@gmail.com>
@madmath03
Merge branch 'develop' into feature/job_to_bg_worker
fd2df64
@madmath03
Merge branch 'feature/job_to_bg_worker' of github.com:Monogramm/erpne…
ac7b9e4 
…xt_ocr into feature/job_to_bg_worker
@AminovE99
✅ Add test for Validation Error
f75dd9b 
Signed-off-by: Emil <amionv99@gmail.com>
@madmath03 madmath03 merged commit 443a683 into develop Feb 3, 2020
@madmath03 madmath03 deleted the feature/job_to_bg_worker branch February 3, 2020 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@madmath03 madmath03 madmath03 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AminovE99 @madmath03