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

API Review #7

Open
1 of 3 tasks
plfiorini opened this issue Jul 18, 2015 · 4 comments
Open
1 of 3 tasks

API Review #7

plfiorini opened this issue Jul 18, 2015 · 4 comments
Assignees
@plfiorini
Labels
in progress task
Milestone
0.7.0

Comments

@plfiorini
Copy link
Member

Review the API.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@plfiorini plfiorini added this to the Version 0.7.0 milestone Jul 18, 2015
@plfiorini plfiorini added the task label Jul 29, 2015
@plfiorini
Copy link
Member Author

@xiangzhai Anyone willing to help with API review is welcome here :)

@plfiorini
Copy link
Member Author

For example KDE developers

@xiangzhai
Copy link
Contributor

Yes, David Edmundson, who concerned by the blocking calls in AccountsService, is the one ;-)

@xiangzhai
Copy link
Contributor

I will email to plasma-devel mailing list and cc those projects` core developers and maintainers, they will show use what they concern ;-)

plfiorini added a commit that referenced this issue Jul 30, 2015
@plfiorini
Fix D-Bus call to DeleteUser
af33e8a 
DeleteUser doesn't return an object path.

Issue: #7
plfiorini added a commit that referenced this issue Jul 31, 2015
@plfiorini
Make AccountsManager::cacheUser() async
13ed3cb 
Fix potential lock up of the entire desktop session.
Probably we need to make more APIs async, that's a task for #7.

Closes #9
@plfiorini plfiorini self-assigned this Jul 31, 2015
plfiorini added a commit that referenced this issue Sep 9, 2015
@plfiorini
By reference
30f4476 
Issue: #7
plfiorini added a commit that referenced this issue Sep 9, 2015
@plfiorini
Cache users and fix userDelete() signal
dda63aa 
Recycle account objects with a cache and change the userDelete()
signal argument to uid_t.  The account was delete and so we no longer
can access it through D-Bus which means that the UserAccount object
cannot be used.

Issue: #7
@davidedmundson davidedmundson mentioned this issue Oct 10, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@plfiorini plfiorini

Labels
in progress task
Projects
None yet
Milestone
0.7.0
Development

No branches or pull requests
2 participants
@plfiorini @xiangzhai