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

Patch2 #61

Merged
merged 15 commits into from
Nov 24, 2020
Merged

Patch2 #61

merged 15 commits into from
Nov 24, 2020

Conversation

ericgaspar
Copy link
Member

  • Fix NGINX 302 redirect
  • Fix missing upgrade_type in upgrade script
  • Add ynh_smart_mktemp helper

@ericgaspar ericgaspar merged commit 837c394 into testing Nov 24, 2020
@ericgaspar ericgaspar deleted the patch branch November 24, 2020 22:39
lapineige added a commit that referenced this pull request Nov 30, 2020
* Set file permissions after creating the log (#53)

Should fix #49

* Updated to V0.12.0 and Nginx fix (#57)

* Set file permissions after creating the log (#53) (#54)

Should fix #49

Authored-by: tituspijean <tituspijean@outlook.com>

* Updated to version v0.12.0

* Fix images

* Fixed path in nginx

Co-authored-by: lapineige <lapineige@users.noreply.github.com>
Co-authored-by: anmol <anmol@jswan2.com>
Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com>

* Update README.md

* rm -r -> ynh_secure_remove (#59)

* Update issue_template.md

Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com>

* Update openssl dependency

Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com>

* Update _common.sh

- Remove unused script that generates an error
- Remove openssl that is part of PHP as default

* Cleaning up

* Fix linter warnings

* Small typos (#60)

* Patch2 (#61)

- Fix NGINX 302 redirect
- Fix missing upgrade_type in upgrade script
- Add ynh_smart_mktemp helper

* Fix NGINX

* Re-implement automatic post-installation

* Ensure file and directories permissions in upgrade

* Replace all ynh_print_info with ynh_script_progression

* Update check_process

* Use permissions system

Co-authored-by: tituspijean <tituspijean@outlook.com>
Co-authored-by: anmol26s <5068843+anmol26s@users.noreply.github.com>
Co-authored-by: anmol <anmol@jswan2.com>
Co-authored-by: Éric Gaspar <46165813+ericgaspar@users.noreply.github.com>
Co-authored-by: ericgaspar <junk.eg@free.fr>
Co-authored-by: Alexandre Aubin <alex.aubin@mailoo.org>
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.

None yet

1 participant