Skip to content
This repository has been archived by the owner on Nov 14, 2018. It is now read-only.

open web apps #448

Closed
wants to merge 35 commits into from
Closed

open web apps #448

wants to merge 35 commits into from

Conversation

michielbdejong
Copy link
Contributor

I renamed my app from 'Unhosted apps' to 'Open web apps' because it's clearer.

This app is aimed at advanced users who want to experiment with new web technology such as Mozilla's Open Web App manifest format and connecting unhosted web apps to your ownCloud data via the remotestorage protocol. this pull request is the cleaned-up version of #47

@ghost
Copy link

ghost commented Jan 12, 2013

Can one of the admins verify this patch?

<name>*dbname*</name>
<create>true</create>
<overwrite>false</overwrite>
<charset>latin1</charset>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be utf8 or much grief will come upon us ;)

@tanghus
Copy link
Contributor

tanghus commented Jan 17, 2013

I haven't tested it, but besides the Latin1 db collation it looks good and clean 👍

@DeepDiver1975
Copy link
Contributor

@owncloud-bot this is ok to test

@karlitschek
Copy link
Contributor

@michielbdejong What is the state of that? Is this ready to be merged in shipped with ownCloud 5 ?

@michielbdejong
Copy link
Contributor Author

yes, this version is quite crude and the next version is already underway with several improvements, but it works, so i think it would be nice if you include it! :)

many thanks,
Michiel.

@karlitschek
Copy link
Contributor

Cool. Well technically we had feature freeze last friday. ;-) When is the new app ready?

@michielbdejong
Copy link
Contributor Author

i can probably have it ready tomorrow evening

@karlitschek
Copy link
Contributor

Ok. This would be great.

@michielbdejong
Copy link
Contributor Author

Bummer, i was working till 3am yesterday and got the app working. now i updated my install to master, and it looks like the app format has changed. i can't get my app to work on 5.0 atm, i can't even get the template app working, this is a whole new learning curve. for now i will get my app working on stable45 first, so that at least i can use it myself, and people who are willing to stay on 4.5 can use it.

then i will find time soon to migrate it to 5.0, but it is not going to be today, sorry.

@DeepDiver1975
Copy link
Contributor

@michielbdejong
there is a php syntax error:

  [apply] Errors parsing /var/lib/jenkins/jobs/pull-request-analyser-for-apps/workspace/open_web_apps/lib/apps.php
    [apply] PHP Parse error:  syntax error, unexpected '[', expecting ')' in /var/lib/jenkins/jobs/pull-request-analyser-for-apps/workspace/open_web_apps/lib/apps.php on line 40

BUILD FAILED
/var/lib/jenkins/jobs/pull-request-analyser-for-apps/workspace/build.xml:45: apply returned: 255

@michielbdejong
Copy link
Contributor Author

it seems that this works on my machine, but i changed it around so as not to use this funny syntax.

@michielbdejong
Copy link
Contributor Author

this old one is superceded by #1260

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants