App Submission: Budibase

[sharknoon]

App Submission

App name

Budibase

256x256 SVG icon

Gallery images

I have tested my app on:

• umbrelOS on a Raspberry Pi
• umbrelOS on an Umbrel Home
• umbrelOS on Linux VM

[sharknoon]

Unfortunately setting user: "1000:1000" in the docker-compose.yml prevents the app from starting.

[joaovictor-local]

I installed on umbrelOS 1.2.1 (Intel N100).
Thank you, @sharknoon! I already have some use cases for it 😄

[nmfretz]

Fantastic work @sharknoon! This one was fun to play with.

Reviewed and tested. Let's get gallery assets made up.

[nmfretz]

@sharknoon, actually one thing that I think we should do:

Right now the single bind mount is set to ${APP_DATA_DIR}/data. This makes sense because budibase is an all-in-one image with a single container directory to bind to the host. But I think we should bind to ${APP_DATA_DIR}/data/budibase instead. The reason being that this will future proof us in the event that a new container is added outside the all-in-one image in the future. For example, an nginx container or some umbrel specific container that requires a bind mount (I dunno, these aren't the greatest example). That means we could just bind ${APP_DATA_DIR}/data/<new-container> and not worry at all about migrating the budibase data into it's own dedicated directory, or worrying about the order of the bind mounts to avoid conflict.

What do you think?

[sharknoon]

@nmfretz Yes, that is a good foresight. I have changed and tested it 😀

[github-actions]

🎉   Linting finished with no errors or warnings   🎉

Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store.

Please review the linting results below and make any necessary changes to your submission.

Linting Results

Severity	File	Description
ℹ️	budibase/docker-compose.yml	Potentially using unsafe user in service "budibase": The default container user "root" can lead to security vulnerabilities. If you are using the root user, please try to specify a different user (e.g. "1000:1000") in the compose file or try to set the UID/PUID and GID/PGID environment variables to 1000.

Legend

Symbol	Description
❌	Error: This must be resolved before this PR can be merged.
⚠️	Warning: This is highly encouraged to be resolved, but is not strictly mandatory.
ℹ️	Info: This is just for your information.

[nmfretz]

Another app ready to go from the legendary @sharknoon!
Gallery assets: https://github.com/getumbrel/umbrel-apps-gallery/tree/master/budibase

Going live now.