Minimal API

[wolfgang-braun]

Hi,
would you consider merging a minimal API feature branch (get matter, get events, create events with bearer auth)?

Even more minimalistic approach, change selected controllers to check wantsJson:

From this

class MatterController extends Controller
{

    public function index(Request $request)
    {
        $filters = $request->except(
            [
                'display_with',
                'page',
                'filter',
                'value',
                'sortkey',
                'sortdir',
                'tab',
                'include_dead',
            ]
        );

        $matters = Matter::filter(
            $request->input('sortkey', 'id'),
            $request->input('sortdir', 'desc'),
            $filters,
            $request->display_with,
            $request->include_dead
        )->simplePaginate(25);
        $matters->withQueryString()->links(); // Keep URL parameters in the paginator links

        return view('matter.index', compact('matters'));
    }

to

public function index(Request $request)
{
    $filters = $request->except([
        'display_with',
        'page',
        'filter',
        'value',
        'sortkey',
        'sortdir',
        'tab',
        'include_dead',
    ]);

    // Sort & Filter
    $query = Matter::filter(
        $request->input('sortkey', 'id'),
        $request->input('sortdir', 'desc'),
        $filters,
        $request->display_with,
        $request->include_dead
    );

    // Check if JSON is requested
    if ($request->wantsJson()) {
        // Eager load events (and optionally event info)
        $matters = $query->with('events.info')->get();

        return response()->json($matters);
    }

    // Default behavior: paginate for HTML view
    $matters = $query->simplePaginate(25);
    $matters->withQueryString()->links();

    return view('matter.index', compact('matters'));
}

Would this be acceptable as a minimal approach?
(See also adding ->with('events.info') and removing pagination)

Thanks!

[jjdejong]

I see no issue if this doesn't break the default functionality, which apparently it doesn't. Why don't you create a pull request?

[funktionslust]

Thanks! Here’s the pull request: #164

It might be helpful to note in the wiki that basic read access via JSON is now possible. Authentication still requires handling CSRF, and write endpoints aren’t supported yet. If there’s demand, I’m happy to add it.