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.

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

WIP: Kimai2 conversion #8

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

WIP: Kimai2 conversion #8

wants to merge 8 commits into from

Conversation

bastilimbach
Copy link
Owner

@bastilimbach bastilimbach commented Sep 23, 2018

Work in progress PR for the conversion of the legacy API.

Login

  • Change placeholders
  • Convert authentication flow
    • ✅Check Kimai Endpoint
    • ✅Check credentials
    • ✅Display appropriate error messages
  • Change demo credentials WIP: Kimai2 conversion #8 (comment)
  • Check version

Timesheet

  • Get timesheet records for user. Blocking Kimai issue: #315

@bastilimbach bastilimbach self-assigned this Sep 23, 2018
let user = User(userName: userName, apiEndpoint: apiEndpoint, apiKey: result.apiKey)
switch result {
case .success(let ping):
if ping.message == "pong" {
Copy link
Owner Author

@bastilimbach bastilimbach Sep 23, 2018

Choose a reason for hiding this comment

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

Change to: ping.version >= 0.1

@bastilimbach
Copy link
Owner Author

@kevinpapst is it possible to add a default API password to all demo users of the demo instance? Then I could use this instance with the user "john_user" for everybody who wants to test Timeu without the need to install their own Kimai instance.

@kevinpapst
Copy link

That is already the case, use api_kitten - see: https://github.com/kevinpapst/kimai2/blob/master/src/DataFixtures/UserFixtures.php#L29

@bastilimbach
Copy link
Owner Author

The login now works with the new /api/version endpoint. If issue #315 is closed, I can work on the rest.

@kevinpapst
Copy link

Nice! I'll will work on that directly after finishing all currently open PRs.

@kevinpapst
Copy link

Sorry, I had to change the version to string, otherwise something like "0.6.1" wouldn't be possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants