Chore: migrate to remote service without manual service#79

Closed

Schmarvinius wants to merge 34 commits into

review-fixesfrom

chore/migrate-to-remote-service

Commits on Jun 16, 2026

Separate the predictionRow from the contextRows and add a constant for [PREDICT]
lisajulia
committed
Add comment on why we the @FunctionalInterface annotation is useful
lisajulia
committed
Add comment to FioriRecommendationHandler
lisajulia
committed
Moved everthing RPT-1 specific into RptInferenceClient
lisajulia
committed
Rename RptInferenceClient.api -> RptInferenceClient.rpt and extract code that creates sdkRow to separate method
lisajulia
committed
Moved everthing RPT-1 specific into RptInferenceClient
lisajulia
committed
Replace MANAGED_FIELDS set with annotation-driven exclusion: @Core.Computed, @readonly and remove logic from 'toSdkRow' method that was already executed elsewhere
lisajulia
committed
Change level of log statement when no suitable context columns are found and recommendations are therefore skipped
lisajulia
committed
Move 'return early if predictRow == null' to earlier in afterRead of FioriRecommendationHandler
lisajulia
committed
Extract 'SAP_Recommendations' into a constant
lisajulia
committed
Get Persistence Service db via Dependency Injection
lisajulia
committed
Move missingPredictionElementNames to earlier in the afterRead of the FioriRecommendationHandler
lisajulia
committed
Add comment about conversion from List<Row> to List<CdsData>
lisajulia
committed
Change check for active Entity: only return early if isActiveEntity is selected and false
lisajulia
committed
Add comments to MockRecommendationClient
lisajulia
committed
Change type of slectcolumns to Set
lisajulia
committed
Add test: row for which we want to do predictions is automatically excluded in the select query returned by buildContextQuery
lisajulia
committed
Add comment about ordering in the select query returned by buildContextQuery
lisajulia
committed
Add comment computeSyntheticKey method and add test
lisajulia
committed
Make sure we dont react on @cds.odata.valuelist : false and add a test for that
lisajulia
committed
Simplify filters in computeContextColumn
lisajulia
committed
Use any single key as RPT-1 index column, not just fields called ID and add test
lisajulia
committed
Adjust sample to changes in cds-feature-recomendations
lisajulia
committed
Add comment about books entity in test model
lisajulia
committed
Update javadoc
lisajulia
committed
Do not register FioriRecommendationHandler if no PersistenceService is found
lisajulia
committed
In RptInferenceClient:resolveIndexColumn: fall back to synthetic index column for non-string single keys
lisajulia
committed
Add checks for the presence of keyNames
lisajulia
committed