Skip to content
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

Fix domain persistence in docker_compose_domains #4327

Open
wants to merge 2 commits into
base: next
Choose a base branch
from

Conversation

vishwamartur
Copy link

@vishwamartur vishwamartur commented Nov 18, 2024

Related to #4326

Update domain persistence in docker_compose_domains field after redeployment.

  • app/Livewire/Project/Application/General.php
    • Add validation for docker_compose_domains in the rules and validationAttributes arrays.
  • app/Models/ApplicationPreview.php
    • Add validateDockerComposeDomainsFormat method to validate the format of docker_compose_domains.
  • app/Http/Controllers/Webhook/Bitbucket.php
    • Add validateDockerComposeDomainsFormat method to validate the format of docker_compose_domains.
  • app/Http/Controllers/Webhook/Gitea.php
    • Add validateDockerComposeDomainsFormat method to validate the format of docker_compose_domains.
  • app/Http/Controllers/Webhook/Github.php
    • Add docker_compose_domains to the data array when creating a new ApplicationDeployment.

/claim #4326

Related to coollabsio#4326

Update domain persistence in `docker_compose_domains` field after redeployment.

* **app/Livewire/Project/Application/General.php**
  - Add validation for `docker_compose_domains` in the `rules` and `validationAttributes` arrays.
* **app/Models/ApplicationPreview.php**
  - Add `validateDockerComposeDomainsFormat` method to validate the format of `docker_compose_domains`.
* **app/Http/Controllers/Webhook/Bitbucket.php**
  - Add `validateDockerComposeDomainsFormat` method to validate the format of `docker_compose_domains`.
* **app/Http/Controllers/Webhook/Gitea.php**
  - Add `validateDockerComposeDomainsFormat` method to validate the format of `docker_compose_domains`.
* **app/Http/Controllers/Webhook/Github.php**
  - Add `docker_compose_domains` to the data array when creating a new `ApplicationDeployment`.

/claim coollabsio#4326
@peaklabs-dev peaklabs-dev changed the base branch from main to next November 19, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant