-
-
Notifications
You must be signed in to change notification settings - Fork 26
The add button is not disabled #149
Comments
Thank you 👍 |
Can you please assign this to me, I can work on this. |
Sure assigned to you @PadamSinha2511 |
I noticed that the compose.yaml file does not have a volume for the app to sync the local development to the container running the application should I add a volume and mount the app directory as volumes: |
This could be the issue that @Bashamega has. @Pradumnasaraf could you let us know if this is missing or if we are we using another way to achieve this? Thank you |
Hey Eddie, Sure, @PadamSinha2511 you can add. I was going to implement the compose watch feature instead of bind mount, but realised, that many people don't have the latest version of compose and it won't work, so I didn't implement that. The issue bind mounts it that it didn't work well many times, especially with Next.js apps. Pradam, do raise a separate PR for that. |
Thank you @Pradumnasaraf 👍 @PadamSinha2511 please can you raise another issue for the docker compose changes |
Sure, here is the issue id #167 |
Description
Hello:)
I have noticed that without entering anything in the GitHub URL placeholder the button is still clickable. It should first test against a regex to see if there is a valid GitHub URL.
Screenshots
No response
Additional information
If you are stuck and don't know the correct regex this should work:
The text was updated successfully, but these errors were encountered: