-
Notifications
You must be signed in to change notification settings - Fork 32
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
Request: Add development support #11
Comments
Hi, thanks for your interest. I'd suggest to start with the high level architecture of the different components. Also, is there any specific feature or bug you'd like to work on and I can give you more details on it. |
Hey Lyang, Thanks for your reply, I was able to run and edit the Lapdev-Dashboard & Api, but I am unable to run the entire project in a local environment |
Lapdev-Dashboard & Api is all you needed to run to add oauth provider support. Are you running dashboard via Trunk? |
I tried this myself and got burnt out. Biggest pain point: running lapdev shouldn't require background systemctl service in order to work. It makes things complicated. Docker has the docker.service, but it also has dockerd, which doesn't exit immediately, so you can see what's going on. This project would benefit a lot from a straightforward way to build it and develop on top of it. Right now it's very unlikely to attract contributors due to this alone, in my opinion. |
We'll improve our devcontainer config in Lapdev repo, so that development is easier. (It should be since one of the purpose of Lapdev is to make on boarding easier.) |
Hey,
I want to contribute to this project, please add some instructions or point me in the right direction.
Many thanks for considering my request.
The text was updated successfully, but these errors were encountered: