WIP Use /api/users endpoint instead of /api/identities for fetching list of users #908
Conversation
d483409
to
6d96d93
Compare
@@ -15,7 +15,7 @@ import { User } from '../models/user'; | |||
export class UserService { | |||
private headers = new Headers({'Content-Type': 'application/json'}); | |||
private userUrl = process.env.API_URL + 'user'; // URL to web api | |||
private identitiesUrl = process.env.API_URL + 'identities'; // URL to web api | |||
private usersUrl = process.env.API_URL + 'users'; // URL to web api |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this end point exist?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not yet. And as I said in the description (see above ^) please do NOT merge until fabric8-services/fabric8-wit#692 is merged ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, sorry I missed that. My bad.
@@ -30,7 +30,7 @@ export class UserService { | |||
} else { | |||
logger.log('UserService running in production mode.'); | |||
} | |||
logger.log('UserService using user url ' + this.userUrl + ' identity url ' + this.identitiesUrl); | |||
logger.log('UserService using user url ' + this.userUrl + ' users url ' + this.usersUrl); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need the logger at this point?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know but I didn't remove the original one. Just updated it accordingly.
6d96d93
to
0f43de9
Compare
fabric8-services/fabric8-wit#692 has been merged. This one should be merged now too. |
[test] |
1 similar comment
[test] |
Another PR is raised with the same changes and fixed tests - #930 |
Fixes fabric8-services/fabric8-wit#672