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.
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
Issue approving timesheet #536
Closed
DanielGilchrist
wants to merge
38
commits into
waynerobinson:payroll-merged
from
ghiculescu:dan-payroll-test
Closed
Issue approving timesheet #536
DanielGilchrist
wants to merge
38
commits into
waynerobinson:payroll-merged
from
ghiculescu:dan-payroll-test
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
Remove unnecessary model UPDATE: add permissions to UK Address Model UPDATE: Moved the UK Address model into payroll UPDATE: Add UkAddress to payroll application UPDATE: User correct constant name UPDATE: Trying to fix uk address UPDATE: It should all be in model? UPDATE: Cleanup uk address from payroll module UPDATE: Include both models UPDATE: Rename payroll module address model UPDATE: Fix rename UPDATE: Changed name to UK Home Address UPDATE: Updated file names UPDATE: Updated file name testing something UPDATE: Include payroll module testing name UPDATE: Moved required fields into existing models UPDATED: Fix constant name UPDATE: Include required UK address fields UPDATE: Add required uk address fields to pr emp Update: Cleanup address line 1 type UPDATE: Update api names UPDATE: Update uk address api names UPDATE: Another attempt UPDATE: Try differnent name formatting UPDATE: Still trying UPDATE: Try with address instead of home address UPDATE: Rename to UkHomeAddress UPDATE: Trying address
UPDATE: Force to json if address UPDATE: Testingg UPDATE: Testing2 UPDATE: Testing attr runs twice UPDATE: Test debugger UPDATE: Trying to see what is happening UPDATE: Testing output UPDATE: Remove ai acalls show me the hash before and after UPDATE: Testing hardcoded address remove hardcode UPDATE: Create address in to_json call UPDATE: Cleanup address hash UPDATE: Cleanup address field
Merge Important OAuth2 changes from upstream
Hotfix/add accounting scopes
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.
No description provided.