D12188: Save and delete a user

[Rishabh Gupta]

rishabhg added inline comments.

INLINE COMMENTS

> jjazeix wrote in AddUpdateUser.qml:46
> they are all strings?

right, they are. 
Missed it some how

> jjazeix wrote in AddUpdateUser.qml:54
> empy log?

ah! mistake

> jjazeix wrote in AddUpdateUser.qml:282
> why do you need an empty user?
> If it is for: "NOTE: The first user created does not get saved until the focus from 'Date of birth' column is explicitly changed. Could not figure the reason for this", we should find the reason and fix it (force the focus change?)

> why do we need an empty user ?

We need to have something in our model so that our delegates get displayed. Maybe a better solution exists 
I'll try something on my end.

> If it is for: "NOTE: The first user created does not get saved until the focus from 'Date of birth' column is explicitly changed. Could not figure the reason for this",

No. Also, just found out that this is the case for every user that's created.

> we should find the reason and fix it (force the focus change?)

forcing the focus change solves the problem.

REVISION DETAIL
  https://phabricator.kde.org/D12188

To: rishabhg, jjazeix, timotheegiet
Cc: #kde_edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180418/dc6e6316/attachment-0001.html>