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

Kill all */ajax/*.php routes and move them to testable controllers #13007

Closed
DeepDiver1975 opened this issue Dec 22, 2014 · 9 comments
Closed

Comments

@DeepDiver1975
Copy link
Member

DeepDiver1975 commented Dec 22, 2014

No description provided.

@DeepDiver1975 DeepDiver1975 added this to the 8.1-next milestone Dec 22, 2014
@LukasReschke LukasReschke self-assigned this Dec 26, 2014
@PVince81
Copy link
Contributor

PVince81 commented Jan 9, 2015

@LukasReschke LukasReschke removed their assignment Jan 18, 2015
@PVince81
Copy link
Contributor

PVince81 commented Feb 4, 2015

@DeepDiver1975 DeepDiver1975 modified the milestones: 8.2-next, 8.1-current Mar 2, 2015
@PVince81
Copy link
Contributor

On-going effort. Feature freeze now, so moving it to 9.0

@PVince81 PVince81 modified the milestones: 9.0-next, 8.2-current Sep 21, 2015
@PVince81
Copy link
Contributor

@Xenopathic do we still have any ajax/* endpoints in files_external ? If not, please tick the box 😄

@RobinMcCorkell
Copy link
Member

We still do 😞

@PVince81
Copy link
Contributor

We're slowly getting there... moving to 9.1 for the rest.

@PVince81 PVince81 modified the milestones: 9.1-next, 9.0-current Feb 11, 2016
@PVince81 PVince81 modified the milestones: 9.2-next, 9.1-current Jun 1, 2016
@PVince81
Copy link
Contributor

PVince81 commented Jun 1, 2016

Ongoing... moving to 9.2.

But this is probably something that can be done in combination with other work touching these areas.

For example for the versions app, implementing a public API for it: #10571 and same for trashbin #15646

@PVince81
Copy link
Contributor

PVince81 commented Nov 9, 2016

@PVince81
Copy link
Contributor

with Phoenix all of these will need to move to public APIs

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

No branches or pull requests

6 participants