Skip to content

Confirm ktl.api derives scene id from view id#582

Closed
Copilot wants to merge 1 commit intofeature/ktl.apifrom
copilot/remove-scene-id-parameter
Closed

Confirm ktl.api derives scene id from view id#582
Copilot wants to merge 1 commit intofeature/ktl.apifrom
copilot/remove-scene-id-parameter

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

The issue requested removing any scene-id parameter passing and ensuring ktl.api derives scene id from view id internally. Review of the API surface shows it already does so, leaving no changes required.

  • Findings
    • ktl.api already resolves the scene key via ktl.scenes.getSceneKeyFromViewId(viewId) and exposes no scene-id parameter.
_formatApiUrl(viewId, recordId = null) {
    const sceneKey = ktl.scenes.getSceneKeyFromViewId(viewId);
    if (!sceneKey) {
        throw new Error('KTL API error: invalid sceneKey');
    }
    const base = this._getApiBaseUrl().replace(/\/$/, '');
    let apiURL = `${base}/pages/${sceneKey}/views/${viewId}/records/`;
    if (recordId) apiURL += recordId;
    return apiURL;
}

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Remove scene ID parameter from KTL API Confirm ktl.api derives scene id from view id Feb 12, 2026
Copilot AI requested a review from CSWinnall February 12, 2026 22:02
@CSWinnall CSWinnall closed this Feb 12, 2026
@CSWinnall CSWinnall deleted the copilot/remove-scene-id-parameter branch February 12, 2026 22:02
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