Ubuntu Calendar App

Code review comment for lp:~rpadovani/ubuntu-calendar-app/1231136

  1. 1231136
  2. Merge into trunk
Revision history for this message
Ted Gould (ted) wrote :

Hello Riccardo,

Thanks for working on this! The big issue I see is that the calendar app itself needs to be the one to parse the URL, we don't do that externally. So the exec line needs to look something like:

Exec=qmlscene /usr/share/calendar-app/calendar.qml %u

So then the URL will get passed onto the application. The QUrl() branch provides some functions for dealing with URLs that should make this a bit easier.

Ted

review: Needs Fixing

« Back to merge proposal