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

Refactor of User Action Executor #820

Merged
merged 40 commits into from
Oct 29, 2019
Merged

Refactor of User Action Executor #820

merged 40 commits into from
Oct 29, 2019

Conversation

romaninsh
Copy link
Member

User Action Executor allows you to bind UI elements with User Actions defined in the model.

Conveniently executor will ask user for missing arguments, provide user with a preview, if action supports a preview and will also show the result of action execution.

More information is comming.

@codecov
Copy link

codecov bot commented Oct 22, 2019

Codecov Report

Merging #820 into develop will decrease coverage by 3.23%.
The diff coverage is 32.29%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #820      +/-   ##
=============================================
- Coverage       76.4%   73.17%   -3.24%     
- Complexity      2297     2544     +247     
=============================================
  Files            119      122       +3     
  Lines           5434     5916     +482     
=============================================
+ Hits            4152     4329     +177     
- Misses          1282     1587     +305
Impacted Files Coverage Δ Complexity Δ
src/ActionExecutor/Preview.php 61.53% <ø> (-1.43%) 8 <0> (ø)
src/ActionExecutor/jsArgumentForm.php 0% <0%> (ø) 12 <12> (?)
src/jsCallback.php 88.23% <100%> (+0.35%) 32 <0> (ø) ⬇️
src/jsReload.php 100% <100%> (ø) 6 <1> (+3) ⬆️
src/Loader.php 96% <100%> (+0.34%) 11 <3> (+2) ⬆️
src/ActionExecutor/jsEvent.php 87.09% <100%> (+4.33%) 15 <3> (+2) ⬆️
src/View.php 87.16% <28.57%> (-2.56%) 146 <7> (+7)
src/ActionExecutor/UserAction.php 31.51% <31.51%> (ø) 88 <88> (?)
src/Form.php 76.92% <0%> (-0.33%) 75% <0%> (+3%)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update abeda03...33e42a4. Read the comment docs.

ibelar and others added 9 commits October 22, 2019 10:32
move loader set function to jsRender so we could apply afterSuccess or jsSuccess after setAction is done.
Add jsSuccess and stateContext
- include web storage for ajaxexec
- code refactoring
@ibelar ibelar mentioned this pull request Oct 25, 2019
@codecov
Copy link

codecov bot commented Oct 28, 2019

Codecov Report

❗ No coverage uploaded for pull request base (develop@92a4405). Click here to learn what that means.
The diff coverage is 8.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #820   +/-   ##
==========================================
  Coverage           ?   71.85%           
  Complexity         ?     2446           
==========================================
  Files              ?      122           
  Lines              ?     5931           
  Branches           ?        0           
==========================================
  Hits               ?     4262           
  Misses             ?     1669           
  Partials           ?        0
Impacted Files Coverage Δ Complexity Δ
src/ActionExecutor/Preview.php 61.53% <ø> (ø) 8 <0> (?)
src/TableColumn/Actions.php 94.28% <ø> (ø) 14 <0> (?)
src/ActionExecutor/jsArgumentForm.php 0% <0%> (ø) 12 <12> (?)
src/jsCallback.php 88.23% <100%> (ø) 32 <0> (?)
src/jsReload.php 100% <100%> (ø) 6 <1> (?)
src/ActionExecutor/jsEvent.php 83.87% <100%> (ø) 15 <3> (?)
src/View.php 87.16% <28.57%> (ø) 146 <7> (?)
src/ActionExecutor/UserAction.php 3.95% <3.95%> (ø) 96 <96> (?)
src/Loader.php 92% <66.66%> (ø) 11 <3> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 92a4405...39b6cd2. Read the comment docs.

@romaninsh romaninsh merged commit 988e16a into develop Oct 29, 2019
@romaninsh romaninsh deleted the feature/user-action branch October 29, 2019 20:23
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.

2 participants