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

Command reordering not saving #325

Merged
merged 2 commits into from
Mar 2, 2017

Conversation

emielmolenaar
Copy link
Contributor

@emielmolenaar emielmolenaar commented Mar 1, 2017

Reorder now fails because a POST request is being sent to /commands/reorder and the application only accepts /projects/{project}/commands/reorder due to the prefix in routes/api.php .

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the Contributing Guidelines?
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • Do the TravisCI tests pass?
  • Does the StyleCI test pass?

NOTE: The last 2 are not required to open a PR and can be done afterwards /
while the PR is open.


Description of change

Reorder now fails because a POST request is being sent to /commands/reorder and the application only accepts /projects/{project}/commands/reorder due to the prefix in routes/api.php .

Reorder now fails because a POST request is being sent to /commands/reorder and the application only accepts /projects/{project}/commands/reorder due to the prefix in routes/api.php .
@emielmolenaar emielmolenaar changed the title Fixes command reordering [FIX] Fixes command reordering Mar 1, 2017
@REBELinBLUE
Copy link
Owner

D'oh! thanks I'll get this merged in later today

@codecov
Copy link

codecov bot commented Mar 1, 2017

Codecov Report

Merging #325 into master will decrease coverage by -0.21%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #325      +/-   ##
============================================
- Coverage     86.07%   85.86%   -0.21%     
  Complexity     1054     1054              
============================================
  Files           174      174              
  Lines          3403     3403              
============================================
- Hits           2929     2922       -7     
- Misses          474      481       +7

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 3f984f3...308863f. Read the comment docs.

@REBELinBLUE
Copy link
Owner

Sorry, got distracted last night with Laravel 5.4.14 breaking loads of the tests. I shall merge this tonight before I do anything else

@emielmolenaar
Copy link
Contributor Author

Sure, no problem! Thanks.

@REBELinBLUE REBELinBLUE merged commit 8b1295d into REBELinBLUE:master Mar 2, 2017
@REBELinBLUE REBELinBLUE changed the title [FIX] Fixes command reordering Command reordering not saving Mar 27, 2017
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.

2 participants