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

IBX-3465: Copied vhost templates from docker repo #50

Merged
merged 3 commits into from
Feb 22, 2023

Conversation

glye
Copy link
Contributor

@glye glye commented Aug 12, 2022

Question Answer
JIRA issue IBX-3465
Type improvement
Target Ibexa version v4.5
BC breaks no

We should maintain vhost templates. The docker repo is no longer maintained. As discussed in ibexa/documentation-developer#1704 we can copy them from the docker repo to here, as this PR does.

Future improvement: Make a command for generating vhosts, similar to the bash script we had in eZ Platform: https://github.com/ezsystems/ezplatform/blob/master/bin/vhost.sh

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly (main for features, the oldest supported for bugs).
  • Ran PHP CS Fixer for new PHP code (use $ composer fix-cs).
  • Asked for a review (ping @ibexa/engineering).

@glye
Copy link
Contributor Author

glye commented Aug 12, 2022

Test failures are irrelevant (phpstan plugin blocked by config)

@adamwojs
Copy link
Member

Small remark: can we change ez_ prefix to ibexa_ in file names ?

@Nattfarinn
Copy link
Contributor

@glye Can you rebrand them from ez to ibexa?

@glye glye force-pushed the ibx3465_added_vhost_templates branch from 609c23d to 3f65666 Compare November 11, 2022 15:16
@glye
Copy link
Contributor Author

glye commented Nov 11, 2022

Rebranded from ez to ibexa. Rebased on updated main branch, because whoops, 10 weeks have passed.

@glye glye force-pushed the ibx3465_added_vhost_templates branch from 3f65666 to 41d22a7 Compare February 22, 2023 09:22
@glye glye merged commit f683dc0 into main Feb 22, 2023
@glye glye deleted the ibx3465_added_vhost_templates branch February 22, 2023 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants