Review Request 110617: Implemented Open Action for Khipu

Punit Mehta punit9461 at gmail.com
Tue Jun 4 09:59:18 UTC 2013



> On May 30, 2013, 2:45 p.m., Aleix Pol Gonzalez wrote:
> > I don't want these reviews open for much longer. Can you open a punit/gsoc2013 branch in Khipu with these patches and commit them there? this way you can move forward.
> 
> Punit Mehta wrote:
>     But I dont have the developer rights. So , I guess I cant push my changes there.!
> 
> Aleix Pol Gonzalez wrote:
>     Then just ask for it:
>     http://techbase.kde.org/Contribute/Get_a_Contributor_Account
>     
>     You can use me a referral, mention that you're a GSoC student.
> 
> Punit Mehta wrote:
>     Thanks! I have submitted the developer application.
> 
> Percy Camilo Triveño Aucahuasi wrote:
>     Aleix, excellent idea, yes I'm glad Punit can take more active part in the development through a new branch. Please, wait until this weekend, I will notify once the branch is created.
>     
>     Percy
>
> 
> Percy Camilo Triveño Aucahuasi wrote:
>     Done, I've created the gsoc2013 branch (this branch doesn't track master since I'm not going to code until the GSoC end) Punit you have developer rights now (there was 2 users associated with you, contact the admin if was an error)
>     
>     Percy
> 
> Aleix Pol Gonzalez wrote:
>     I was expecting Punit to create this branch...
> 
> Percy Camilo Triveño Aucahuasi wrote:
>     Oh I have misinterpreted; but I know there isn't any problem, the good news are that Punit is already uploading his work into the repo.
>     
>     Percy

Yes..! Percy has already suggested me that he is going to create the branch. So , I thought not to interfere in his work ! :) 
By the way , I have already pushed most of the code of my local branch into gsoc2013 remote Khipu branch. Now , I am still working to improve it.


- Punit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110617/#review33438
-----------------------------------------------------------


On May 30, 2013, 2:40 p.m., Punit Mehta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110617/
> -----------------------------------------------------------
> 
> (Updated May 30, 2013, 2:40 p.m.)
> 
> 
> Review request for KDE Edu, Aleix Pol Gonzalez and Percy Camilo Triveño Aucahuasi.
> 
> 
> Description
> -------
> 
> The diff contains the implementation of the open action in Khipu.! The attached parent diff is for the save action which is already reviewed.! 
> If the user saves the plotspaces in .khipu file and then  he can open the file when he wants.! The initial implementation is done in a way that it considers the case only when on work is done ( i.e the user has just opened the application and then in the very next step , he is going to open his previously saved .khipu file.! I am working for dealing with the other cases also..! 
> 
> - I have added the entry of plot-colot in the .khipu file as we are requesting and creating the plots when the plotspace-thumbnail is doubleclicked.! So , we need that information also.!
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt f7dc583 
>   src/CMakeLists.txt 0a50c54 
>   src/dashboard.h a2a9c46 
>   src/dashboard.cpp 688a49d 
>   src/datastore.h a1276a7 
>   src/datastore.cpp 616e0cc 
>   src/mainwindow.h 1f059f8 
>   src/mainwindow.cpp 9d6e010 
>   src/plotseditor.h 47304c6 
>   src/plotseditor.cpp 106cdf5 
> 
> Diff: http://git.reviewboard.kde.org/r/110617/diff/
> 
> 
> Testing
> -------
> 
> Done.! as mentioned , it works very well for the specified case.!
> 
> 
> Thanks,
> 
> Punit Mehta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20130604/e775e16b/attachment.html>


More information about the kde-edu mailing list