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

feat: ignore known erpnext exceptions #29

Closed
wants to merge 23 commits into from
Closed

feat: ignore known erpnext exceptions #29

wants to merge 23 commits into from

Conversation

karthikeyan5
Copy link
Contributor

Adding config to ignore certain known ERPNext Exceptions.

  1. No Employee found for the given employee User ID in the Biometric device.
  2. Employee is inactive for the given employee User ID in the Biometric device.
  3. Duplicate Employee Checkin found. (This exception can happen if you have cleared the logs/status.json of this script) fixes Remove duplicated timestamp for employee #26

The above errors can now be ignored.

karthikeyan5 and others added 23 commits May 31, 2022 19:01
@karthikeyan5
feat: ignore known erpnext exceptions
7790f55
@Anurag810 @karthikeyan5
fix: Important Validation and UI/UX
0ac6e5c
@surajshetty3416 @karthikeyan5
fix: UI changes
c706406
@Anurag810 @karthikeyan5
feat: UI/UX Enhancement
9c1722b
@gavindsouza @karthikeyan5
chore: start daemon instead of os.system
c613445
@gavindsouza @karthikeyan5
fix: daemonized service for device sync
f286849
@gavindsouza @karthikeyan5
refactor: biometric_easy_installer file
c3f35eb 
    * style: tabs to spaces
    * DRYier code
    * fixed validations
@Anurag810 @gavindsouza @karthikeyan5
feat: start/stop sync service
8f04dd3 
Co-authored-by: Gavin D'souza <gavin18d@gmail.com>
@Anurag810 @gavindsouza @karthikeyan5
fix: dont allow window close if services running
5e39f67 
chore: update validation messages

Co-authored-by: Gavin D'souza <gavin18d@gmail.com>
@gavindsouza @karthikeyan5
chore: unused imports and absolute python path added
9c47aad
@gavindsouza @karthikeyan5
fix: rename files by context
053e938
@gavindsouza @karthikeyan5
docs: update README
117814b
@gavindsouza @karthikeyan5
docs: add resources section
c34dfd0
@gavindsouza @karthikeyan5
fix: split command before feeding to Popen object
902860a
@gavindsouza @karthikeyan5
chore: print exceptions raised in main
3c9dec8
@gavindsouza @karthikeyan5
fix: kill background process instead of terminating it
f0c4ea7
@KaviyaPeriyasamy @karthikeyan5
fix(multiple-entries):grouping of common shifts device id together.
5a9f6b5
@KaviyaPeriyasamy @karthikeyan5
Creation of executable files commands are updated in Readme.
381250b
@KaviyaPeriyasamy @karthikeyan5
readme updated
b0e3bd5
@KaviyaPeriyasamy @karthikeyan5
commands updated for windows
72b1385
@KaviyaPeriyasamy @karthikeyan5
show logs in gui
9497054
@KaviyaPeriyasamy @karthikeyan5
safe date conversion
da8f1ff
@KaviyaPeriyasamy @karthikeyan5
update error and normal logs in GUI based on the start date
422d14f
@karthikeyan5
Copy link
Contributor Author

opened #30, this PR Branch was not rebased properly.

@karthikeyan5 karthikeyan5 deleted the handle_erpnext_exceptions branch May 31, 2022 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove duplicated timestamp for employee
5 participants
@karthikeyan5 @Anurag810 @surajshetty3416 @gavindsouza @KaviyaPeriyasamy