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.

Sign up for GitHub

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

Jump to bottom

merge #349 scripting hooks #374

Merged
merged 31 commits into from Jan 7, 2018
Merged

merge #349 scripting hooks #374

merged 31 commits into from Jan 7, 2018

Conversation

syjer
Copy link
Member

@syjer syjer commented Jan 7, 2018

Obviously it's not the complete task, but most of it.

syjer and others added 29 commits December 27, 2017 19:20
@syjer
#349 stubs/initial work
6d127dc
@syjer
#349 add support for inserting/updating/deleting scripts
5dd972a
@cbellone
suppress unchecked warnings, prevent NPE
c95fe35
@syjer
#349 refactor script: remove configuration column, add extension mana…
7c487c7 
…ger + stubs for calling the plugins
@syjer
#349 extension manager: call the async script service
4f8f850
@syjer
#349 wire up async script execution
29cda57
@syjer
#349 unused import
1bf85f1
@syjer
#349 mysql/pgsql scripts
bd88745
@syjer
#349 mysql: fix sql scripts
6fd3cf5
@syjer
use JSON.stringify/GSON.fromJSON to simplify the java<->js interop
02df411
@syjer
#349 add missing fk
57ad6cd
@syjer
#349 add hook for handling invoice generation scripts
016c05b
@syjer
#349 add hook for handling invoice generation scripts, use optional
5000ecf
@syjer
#349 fix test
fb567ee
@syjer
#349 add sample
5593e94
@syjer
#349 keep the order of active scripts always the same when loading them
6864a61
@cbellone
#349 - UI refinements - first draft
2526c22
@syjer
#349 refactor script -> extension
1409ba0
@syjer
#349 support global script too
75fd9b3
@syjer
#349 refactor: deleteReservations -> deleteReservation, add new event…
a3b0448 
…s for the extensions
@cbellone
#349 - UI refinements
9f906e2
@cbellone
#349 - UI refinements
ffdcff0
@cbellone
#349 - UI refinements
8fe25e4
@cbellone
#349 - split
f58811f
@syjer
#349 fix ui
20b62c9
@syjer
#349 switch from . to - as a path separator, handle correctly update,…
18a9af8 
… fix ui
@syjer
#349 wire up RESERVATION_EXPIRED and RESERVATION_CANCELLED events
9088ddd
@cbellone
#349 - display event name
ba05cc7
@cbellone
#349 - rename events
663ecc9
@syjer syjer requested a review from cbellone January 7, 2018 19:15
@coveralls
Copy link

coveralls commented Jan 7, 2018 •
edited

Coverage Status

Coverage increased (+0.7%) to 50.819% when pulling ba05cc7 on 349-scripting-hooks into 6dc448d on master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 50.819% when pulling ba05cc7 on 349-scripting-hooks into 6dc448d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 50.819% when pulling ba05cc7 on 349-scripting-hooks into 6dc448d on master.

@coveralls
Copy link

coveralls commented Jan 7, 2018 •
edited

Coverage Status

Coverage increased (+0.7%) to 50.819% when pulling ba05cc7 on 349-scripting-hooks into 6dc448d on master.

@coveralls
Copy link

coveralls commented Jan 7, 2018 •
edited

Coverage Status

Coverage increased (+0.7%) to 50.81% when pulling 5b45644 on 349-scripting-hooks into 6dc448d on master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 50.81% when pulling 5b45644 on 349-scripting-hooks into 6dc448d on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 50.81% when pulling 5b45644 on 349-scripting-hooks into 6dc448d on master.

@coveralls
Copy link

coveralls commented Jan 7, 2018 •
edited

Coverage Status

Coverage increased (+0.7%) to 50.81% when pulling 5b45644 on 349-scripting-hooks into 6dc448d on master.

@cbellone cbellone merged commit 49c0b5c into master Jan 7, 2018
@syjer syjer deleted the 349-scripting-hooks branch February 16, 2018 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cbellone cbellone cbellone approved these changes

Assignees

@cbellone cbellone

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@syjer @coveralls @cbellone