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

Add post hooks for apps operation #360

Merged
merged 2 commits into from Feb 8, 2018

Conversation

Projects
None yet
3 participants
@zamentur
Contributor

zamentur commented Aug 28, 2017

Problem

Some apps are dependent of others apps (vpnclient&hotspot, baikal&agendav, coin&openvpn ...).
There is a lot of scenario where an app could want do an action if another app is installed, removed, upgraded or change url.

Suggested solution

Add this for hooks

I have set up some env var like recommended in this ticket : https://dev.yunohost.org/issues/818

The var are the same than var sending to the scripts/ACTION

How to test

/etc/yunohost/hooks.d

.
├── post_app_change_url
│   └── youpi
├── post_app_install
│   └── youpi
├── post_app_remove
│   └── youpi
└── post_app_upgrade
    └── youpi

youpi

#!/bin/bash
printenv | grep YNH >> /root/yolo

Like that you can see in the yolo file which variable can be used

PR Status

Opinion needed

Validation

  • Principle agreement 0/2 :
  • Quick review 0/1 :
  • Simple test 0/1 :
  • Deep review 0/1:
@Psycojoker

This comment has been minimized.

Show comment
Hide comment
@Psycojoker

Psycojoker Oct 6, 2017

Member

As anyone tested it?

Member

Psycojoker commented Oct 6, 2017

As anyone tested it?

@alexAubin alexAubin added this to the 2.7.x (fixes) or 2.8 milestone Oct 7, 2017

@Psycojoker Psycojoker merged commit c88cafb into unstable Feb 8, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@Psycojoker Psycojoker deleted the enh-add-post-hook-for-apps branch Feb 8, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment