-
Notifications
You must be signed in to change notification settings - Fork 60
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
Use json instead of csv #197
Comments
Quick update : finished converting all csv files to json, was the hardest task till now :p i will have to check every file if the json syntax is correct. once everything is okay, i will update the script which shouldn't be complicated |
Great, I might have some freetime this weekend, if there's still something left to do then :P |
I have some free time today, i will see if i can fix everything and release a new version ! :P |
@wa4557 it's working pretty well now, i just need to modify the revert code. I've removed the google chrome notification center support as it's removed in the latest versions of chrome, and it was causing too many issues, and a lot of lines codes...People will just have to upgrade their browser ;) |
Looking good to me. Wasn't able to find any breakage so far. Will have a look into the code a bit later... EDIT: One error
EDIT2: Can we remove the chrome scripts? Can we outsource the dropbox stuff to a seperate script (a more modular approach) |
I will see how to fix the issue. We still need chrome script, as i've only removed the icons that were used by the notification center; But chrome still have a tray icon, the get_correct_chrome_icons were used by the notification center, also the chrome images we had in the database folder. I don't know how we can really fix the dropbox issue :( if you have any idea feel free to tell me about it 👍 |
Moved the dropbox code to a new script "dropbox", and added a new key to the json files, that have a script that will be executed on the application path. |
I was wondering where the CSV database file was. The documentation on how to contribute still refers to that, but this clears it up. |
@rpbaptist i need to modify the documentation as too many things changed since the latest version... but the new code is more easy to understand than before ;) |
As discussed here #196
I think it's time to move to a more readable database by us and a way to make adding new icons more easier.
I started moving some database files in the json branch, if you have some free time a help will be much appreciated. Once all the files are using the new format, i will modify the script to read the json files instead of the csv files. Shouldn't be hard, but i will try to make all the stuff use the json file and we won't have to treat each application differently on the python script (reduce the number of the scripts).
CC: @wa4557
The text was updated successfully, but these errors were encountered: