Time application #5
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…, but still end the shift, if the lane is unable to contact the server. Now it dies gracefully and does not end the shift.
Author: Seth McClain <smcclain@montana.com> Committer: root <root@lane>
|
Do you have a db dump that you could share? Maybe as a gist? I was able to pull your commits to a dev branch, but really don't know where to begin checking out the application. Regardless, once you're comfortable with the code that is in your repo, we should pull this into the v2.3_dev branch using pull & merge instead of cherry-pick. |
|
I have matured the code a lot. Once I get this new code into github, a data dump won't be necessary. You should be able to just start scheduling existing members. |
|
I completely forgot to upload the code last night for the time keeper. Shame on me. Hopefully soon. I plan on making it up there tomorrow, time permitting, to get some shopping done. Hopefully I'll have time then. |
…ition of the email field for the members tables.
…passed the intended end time.
…ight or zero, in the processing of that time the hour would get converted to '0' because the parseInt was always taking '08' as being decimal.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I finally got around to adding the time application. I have more work on it than this, but I'll get that added later in due time. It is almost completely done.