Skip to content

latest changes #84

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.

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

Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

latest changes #84

wants to merge 49 commits into from

Conversation

the0ne
Copy link

@the0ne the0ne commented Aug 21, 2018

added web ui at /index.php (note: android app endpoint needs to end with .../api.php)
added synced "checked" flag (strike-through)
added local config file support to avoid overwriting local settings on update
added support for qr-code scanning and ean->product resolution
etc.

please test it out thoroughly.

jklmnn and others added 30 commits September 23, 2015 15:15
QR-Code (base64) generation added
Idea for postinst.sh
Not sure how to automatically find config.php, any ideas?
update file for postinst.sh
Idea for UPDATE.php
SQL Update does not work yet
To check, set BACKEND_VERSION in CONSTANTS.php < 1
correction of installation path and added some info and direct link to the wiki.
Merged
To allow for slite database persistence accross Docker container
destruction/creation, we use a Docker volume mounted on
'/shoppinglist/data'. Files on the volume (ie 'shoppinglist.sqlite') are
of course not served by nginx.
@ghost
Copy link

ghost commented Aug 21, 2018 via email

@the0ne
Copy link
Author

the0ne commented Aug 21, 2018

Don't worry I'm not bored ;-)
But I'm using this piece of software nearly daily and I love it!

@ghost
Copy link

ghost commented Aug 21, 2018 via email

@the0ne
Copy link
Author

the0ne commented Aug 21, 2018

just created the pr for the app as GroundApps/ShoppingList#65

@shervin86
Copy link

Hi,
I am wondering if there is any update on this PR?

@the0ne
Copy link
Author

the0ne commented Apr 17, 2020

Looks like nobody has tested or reviewed it yet.

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

Successfully merging this pull request may close these issues.

8 participants