-
Notifications
You must be signed in to change notification settings - Fork 11
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
automatically create new category when category in integrations is not found #11
Conversation
Follow up to your comment in pawelmalak#7 : The way I first implemented the app categories for docker/kubernetes integrations assumed you already had categories set up before hand, which was my case since the integrations weren't in Flame when I initially forked it. I have made a few changes to permanently add categories (they will be saved to the database, unlike docker/kubernetes apps) found in docker/kubernetes. If custom order is enabled, those categories will be added at the end, but can be re-ordered manually afterwards. You can try those changes by using the |
Yes, seems to work fine - without removing anything (I plan to do that, just because, but it was nice to test without). In CLI you do: Thanks! |
Can you try the new build? Also, make sure you are on the home page, and not on "/applications", since that page always shows all categories, even if no pinned items are in it. |
I am on the home page, but unfortunately, it still shows a "Docker" category title, with no apps below. |
The fix applied only to non-default categories. I have now applied it to non-default categories as well. |
Works. Thanks alot! :) |
Good to hear! This should be available in the |
…t found (#11) automatically create new category when category in docker or kubernetes labels is not found
If you used the same database as the original repository, can you try with a new one? If that works, there may be an issue with the migration from the original repository to my fork. Also, there are currently some broken features in my fork, such as adding bookmarks / bookmark categories. I don't have much time to investigate these problems in the following weeks, but I should be able to take a look into it by the end of November (sorry, I know that's a long time to wait for a fix). |
Thanks for the feedback.....I only don't fully understand what you mean with "new database" as I took your docker container, i.e. I would have expected that it contains the new database. |
The database is saved in the volume you specified, so it should be in To create a new database, you should either move/rename the database file in |
Hello, Got the same problem than @christofkac , i don't use the original container, and got this in the logs
Here are the sqlite database schema CREATE TABLE `SequelizeMeta` (`name` VARCHAR(255) NOT NULL UNIQUE PRIMARY KEY);
CREATE TABLE `config` (`id` INTEGER PRIMARY KEY AUTOINCREMENT, `key` VARCHAR(255) NOT NULL UNIQUE, `value` VARCHAR(255) NOT NULL, `valueType` VARCHAR(255) NOT NULL, `isLocked` TINYINT DEFAULT 0, `createdAt` DATETIME NOT NULL, `updatedAt` DATETIME NOT NULL);
CREATE TABLE sqlite_sequence(name,seq);
CREATE TABLE `weather` (`id` INTEGER PRIMARY KEY AUTOINCREMENT, `externalLastUpdate` VARCHAR(255), `tempC` FLOAT, `tempF` FLOAT, `isDay` INTEGER, `cloud` INTEGER, `conditionText` TEXT, `conditionCode` INTEGER, `createdAt` DATETIME NOT NULL, `updatedAt` DATETIME NOT NULL);
CREATE TABLE `categories` (`id` INTEGER PRIMARY KEY AUTOINCREMENT, `name` VARCHAR(255) NOT NULL, `isPinned` TINYINT DEFAULT 0, `createdAt` DATETIME NOT NULL, `updatedAt` DATETIME NOT NULL, `orderId` INTEGER DEFAULT NULL);
CREATE TABLE `bookmarks` (`id` INTEGER PRIMARY KEY AUTOINCREMENT, `name` VARCHAR(255) NOT NULL, `url` VARCHAR(255) NOT NULL, `categoryId` INTEGER NOT NULL, `icon` VARCHAR(255) DEFAULT '', `createdAt` DATETIME NOT NULL, `updatedAt` DATETIME NOT NULL);
CREATE TABLE `apps` (`id` INTEGER PRIMARY KEY AUTOINCREMENT, `name` VARCHAR(255) NOT NULL, `url` VARCHAR(255) NOT NULL, `icon` VARCHAR(255) NOT NULL DEFAULT 'cancel', `isPinned` TINYINT DEFAULT 0, `createdAt` DATETIME NOT NULL, `updatedAt` DATETIME NOT NULL, `orderId` INTEGER DEFAULT NULL); and this is my docker compose file version: '2.1'
services:
flame:
image: ghcr.io/fdarveau/flame:latest
container_name: flame
volumes:
- /home/me/docker_compose/flame/data_flame:/app/data
ports:
- 80:5005
restart: unless-stopped Did i miss something in my conf ? |
I started working on updating this fork with the latest changes up to v2.1.1. Since there are a ton of changes, including some changes that I had implemented in this fork such as ordering bookmarks, I basically started from scratch to make it easier to maintain on my side by using pawelmalak's code instead of mine for those features. You can check the progress in #12 and you can pull Also, I noticed the Issues section of the repository was disabled, so I enabled it for future issues users encounter with this fork. |
@christofkac @Desaydrone I finished updating this fork to the latest version and everything seems to work fine on my end. If you still need this application, can you give it a try? If you encounter any problem, you can open a new issue to discuss specific problems. Note that there are some breaking changes which require deleting/renaming the database file, as described in the updated README : https://github.com/fdarveau/flame#important-update-2021-12-10. |
Hi, |
Pulled the |
If you are using images from pawelmalak, you will have to open issues in his repository. If you want to try or need features from my fork (such as app categories), you should use |
Hi, I works fine for me now, i can create category for apps. Awesome job, thanks a lot !! |
Hi,
Container starts, but when I click on /apps, I get simply a black screen. I also deleted the persistent volume from my previous installation, so there should not be any left overs. |
I tired on another server to create the container from ghcr.io/fdarveau/flame:latest with the same result. Any help is appreciated. |
What does the logs say? |
@christofkac Found the issue. I accidentally changed one of the links to the applications page to I also added a new "Not found" page to handle that kind of error more gracefully, with a link to the homepage that makes the app work again instead of always being a blank page. If you encounter any other problem, please open a new issue to make things easier to track. |
Cool, works. Thanks. |
No description provided.