Skip to content

Commit

Permalink
Revert "Rename Analytics -> ALP (#1223)" (#1230)
Browse files Browse the repository at this point in the history
This reverts commit 46d2f3a.
@stewsk I had to revert that as that broke tests based on sflight →
let's revisit on Monday
  • Loading branch information
danjoa authored Oct 4, 2024
1 parent 111d765 commit be4a00a
Show file tree
Hide file tree
Showing 37 changed files with 58 additions and 77 deletions.
2 changes: 1 addition & 1 deletion .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ updates:
interval: weekly

- package-ecosystem: npm
directory: /app/travel_alp
directory: /app/travel_analytics
versioning-strategy: increase-if-necessary
schedule:
interval: weekly
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/maven.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
npm run test:java -- --ci
- name: Run integration tests - Analyze Bookings
working-directory: ./app/travel_alp
working-directory: ./app/travel_analytics
run: |
npm ci
npm run test:java -- --ci
2 changes: 1 addition & 1 deletion .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
npm run test:node -- --ci
- name: Run integration tests - Analyze Bookings
working-directory: ./app/travel_alp
working-directory: ./app/travel_analytics
run: |
npm ci
npm run test:node -- --ci
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ mvn spring-boot:run

Open these links in your browser:
* http://localhost:4004/travel_processor/webapp/index.html for processing the travel data
* http://localhost:4004/travel_alp/webapp/index.html for the [Analytical List Page](https://ui5.sap.com/#/topic/3d33684b08ca4490b26a844b6ce19b83) (ALP)
* http://localhost:4004/travel_analytics/webapp/index.html for the [Analytical List Page](https://ui5.sap.com/#/topic/3d33684b08ca4490b26a844b6ce19b83) (ALP)

Log in with user `amy` and empty password.

Expand Down
4 changes: 1 addition & 3 deletions _i18n/i18n.properties
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,8 @@ Name=Name
AirlineNumber=Airline Number
Status=Status
DepartureAirport=Departure Airport
DestinationAirport=Destination Airport
ArrivalAirport=Arrival Airport
Distance=Distance
Airport=Airport
AirportID=Airport

BookingSupplement=Booking Supplement
BookingSupplements=Booking Supplements
Expand Down
4 changes: 1 addition & 3 deletions _i18n/i18n_de.properties
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,8 @@ Name=Name
AirlineNumber=Fluggesellschaftsnummer
Status=Status
DepartureAirport=Abflughafen
DestinationAirport=Zielflughafen
ArrivalAirport=Zielflughafen
Distance=Distanz
Airport=Flughafen
AirportID=Flughafen

BookingSupplement=Buchungszusatz
BookingSupplements=Buchungszusätze
Expand Down
4 changes: 1 addition & 3 deletions _i18n/i18n_en.properties
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,8 @@ Name=Name
AirlineNumber=Airline Number
Status=Status
DepartureAirport=Departure Airport
DestinationAirport=Destination Airport
ArrivalAirport=Arrival Airport
Distance=Distance
Airport=Airport
AirportID=Airport

BookingSupplement=Booking Supplement
BookingSupplements=Booking Supplements
Expand Down
6 changes: 2 additions & 4 deletions _i18n/i18n_fr.properties
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,8 @@ Name=Nom
AirlineNumber=Numéro de compagnie aérienne
Status=Statut
DepartureAirport=Aéroport de départ
DestinationAirport=Aéroport de destination
ArrivalAirport=Aéroport de destination
Distance=Distance
Airport=Aéroport
AirportID=Aéroport

BookingSupplement=Supplément de réservation
BookingSupplements=Réservation de suppléments
Expand All @@ -54,7 +52,7 @@ SupplementType=Type de supplément

Passenger=Passager
LastName=Last Name
PlaneType=Type D'avion
PlaneType=Type d'avion
MaximumSeats=Nombre maximal de places
OccupiedSeats=Places occup.

Expand Down
12 changes: 0 additions & 12 deletions app/labels.cds
Original file line number Diff line number Diff line change
Expand Up @@ -80,11 +80,6 @@ annotate schema.Airline with @title: '{i18n>Airline}' {
CurrencyCode @title: '{i18n>CurrencyCode}';
}

annotate schema.Airport with @title: '{i18n>Airport}' {
AirportID @title: '{i18n>AirportID}' @Common.Text: Name;
Name @title: '{i18n>Name}';
}

annotate schema.Flight with @title: '{i18n>Flight}' {
AirlineID @title: '{i18n>AirlineID}';
FlightDate @title: '{i18n>FlightDate}';
Expand All @@ -96,13 +91,6 @@ annotate schema.Flight with @title: '{i18n>Flight}' {
OccupiedSeats @title: '{i18n>OccupiedSeats}';
}

annotate schema.FlightConnection with {
@Common.Label: '{i18n>DepartureAirport}'
DepartureAirport;
@Common.Label: '{i18n>DestinationAirport}'
DestinationAirport;
}

annotate schema.Supplement with @title: '{i18n>Supplement}' {
SupplementID @title: '{i18n>SupplementID}' @Common.Text: Description;
Price @title: '{i18n>Price}' @Measures.ISOCurrency: CurrencyCode_code;
Expand Down
2 changes: 1 addition & 1 deletion app/services.cds
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ using from './travel_processor/capabilities';
using from './travel_processor/field-control';
using from './travel_processor/layouts';

using from './travel_alp/annotations';
using from './travel_analytics/annotations';
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
using ALPService as service from '../../srv/alp-service';
using AnalyticsService as service from '../../srv/analytics-service';

annotate service.Bookings with @(
Aggregation.CustomAggregate #FlightPrice : 'Edm.Decimal',
Expand Down Expand Up @@ -385,7 +385,7 @@ annotate service.Bookings with @UI : {
// { Value : to_Flight.to_Connection.DepartureAirport.AirportID,
// Label: '{i18n>DepartureAirport}' },
// { Value : to_Flight.to_Connection.DestinationAirport.AirportID,
// Label: '{i18n>DestinationAirport}' },
// Label: '{i18n>ArrivalAirport}' },
// { Value : to_Flight.to_Connection.Distance, },

// Workaround:
Expand Down
File renamed without changes.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"name": "travel-alp",
"name": "travel-analytics",
"version": "1.0.0",
"private": true,
"description": "SFlight ALP",
Expand Down
8 changes: 4 additions & 4 deletions app/travel_alp/ui5.yaml → app/travel_analytics/ui5.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# yaml-language-server: $schema=https://sap.github.io/ui5-tooling/schema/ui5.yaml.json
specVersion: "3.0"
metadata:
name: sap.fe.cap.travel_alp
name: sap.fe.cap.travel_analytics
type: application

resources:
Expand All @@ -17,16 +17,16 @@ builder:
- name: ui5-task-zipper
afterTask: generateCachebusterInfo
configuration:
archiveName: travel-alp
archiveName: travel-analytics
additionalFiles:
- xs-app.json

server:
customMiddleware:
- name: ui5-middleware-simpleproxy
mountPath: /alp
mountPath: /analytics
afterMiddleware: compression
configuration:
baseUri: http://localhost:4004/alp
baseUri: http://localhost:4004/analytics
username: admin # dummy credentials for local testing
password: admin # dummy credentials for local testing
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
sap.ui.define(["sap/fe/core/AppComponent"], function (Component) {
"use strict";
return Component.extend("sap.fe.cap.travel_alp.Component", {
return Component.extend("sap.fe.cap.travel_analytics.Component", {
metadata: { manifest: "json" },
});
});
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
src="https://ui5.sap.com/1.128.0/resources/sap-ui-core.js"
data-sap-ui-theme="sap_horizon"
data-sap-ui-oninit="module:sap/ui/core/ComponentSupport"
data-sap-ui-resourceroots='{ "sap.fe.cap.travel_alp": "./" }'
data-sap-ui-resourceroots='{ "sap.fe.cap.travel_analytics": "./" }'
data-sap-ui-compatVersion="edge"
data-sap-ui-async="true"
data-sap-ui-preload="async"
Expand All @@ -31,9 +31,9 @@
<body class="sapUiBody sapUiSizeCompact" id="content">
<div
data-sap-ui-component
data-name="sap.fe.cap.travel_alp"
data-name="sap.fe.cap.travel_analytics"
data-id="container"
data-settings='{"id" : "sap.fe.cap.travel_alp"}'
data-settings='{"id" : "sap.fe.cap.travel_analytics"}'
data-handle-validation="true"
></div>
</body>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
{
"_version": "1.42.0",
"sap.app": {
"id": "sap.fe.cap.travel_alp",
"id": "sap.fe.cap.travel_analytics",
"type": "application",
"title": "{{title}}",
"description": "{{description}}",
"i18n": {
"bundleName": "sap.fe.cap.travel_alp.i18n.i18n",
"bundleName": "sap.fe.cap.travel_analytics.i18n.i18n",
"supportedLocales": [
"en",
"de",
Expand All @@ -19,7 +19,7 @@
},
"dataSources": {
"mainService": {
"uri": "alp/",
"uri": "analytics/",
"type": "OData",
"settings": {
"odataVersion": "4.0"
Expand Down Expand Up @@ -83,7 +83,7 @@
"i18n": {
"type": "sap.ui.model.resource.ResourceModel",
"settings": {
"bundleName": "sap.fe.cap.travel_alp.i18n.i18n",
"bundleName": "sap.fe.cap.travel_analytics.i18n.i18n",
"supportedLocales": [
"en",
"de",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,11 @@
},
},
applications: {
"sapfecapsflightsflightalp-tile": {
title: "SFlight ALP",
"sapfecapsflightsflightanalytics-tile": {
title: "SFlight Analytics",
description: "SFlight ALP",
additionalInformation:
"SAPUI5.Component=sap.fe.cap.travel_alp",
"SAPUI5.Component=sap.fe.cap.travel_analytics",
applicationType: "URL",
url: "../",
},
Expand All @@ -69,7 +69,7 @@
data-sap-ui-theme="sap_horizon"
data-sap-ui-compatVersion="edge"
data-sap-ui-language="en"
data-sap-ui-resourceroots='{"sap.fe.cap.travel_alp": "../"}'
data-sap-ui-resourceroots='{"sap.fe.cap.travel_analytics": "../"}'
data-sap-ui-frameOptions="allow"
>
// NON-SECURE setting for testing environment
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
data-sap-ui-animation="false"
data-sap-ui-compatVersion="edge"
data-sap-ui-resourceroots='{
"sap.fe.cap.travel_alp": "../../"
"sap.fe.cap.travel_analytics": "../../"
}'
data-sap-ui-theme="sap_horizon"
data-sap-ui-libs="sap.m, sap.fe.core"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
sap.ui.require(
[
"sap/fe/test/JourneyRunner",
"sap/fe/cap/travel_alp/test/integration/OpaJourney",
"sap/fe/cap/travel_alp/test/integration/pages/BookingsList",
"sap/fe/cap/travel_alp/test/integration/pages/BookingsObjectPage",
"sap/fe/cap/travel_analytics/test/integration/OpaJourney",
"sap/fe/cap/travel_analytics/test/integration/pages/BookingsList",
"sap/fe/cap/travel_analytics/test/integration/pages/BookingsObjectPage",
],
function (JourneyRunner, opaJourney, BookingsList, BookingsObjectPage) {
"use strict";

const runner = new JourneyRunner({
// start index.html in web folder
launchUrl:
sap.ui.require.toUrl("sap/fe/cap/travel_alp") + "/index.html",
sap.ui.require.toUrl("sap/fe/cap/travel_analytics") + "/index.html",
});

runner.run(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ sap.ui.define(["sap/ui/test/opaQunit"], function (opaTest) {

return {
run: function () {
QUnit.module("Travel ALP Tests");
QUnit.module("Travel Analytics Tests");

opaTest("Start application", function (Given, When, Then) {
Given.iStartMyApp();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ sap.ui.define(["sap/fe/test/ListReport"], function (ListReport) {

return new ListReport(
{
appId: "sap.fe.cap.travel_alp",
appId: "sap.fe.cap.travel_analytics",
componentId: "BookingsList",
entitySet: "Bookings",
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ sap.ui.define(["sap/fe/test/ObjectPage"], function (ObjectPage) {

return new ObjectPage(
{
appId: "sap.fe.cap.travel_alp",
appId: "sap.fe.cap.travel_analytics",
componentId: "BookingsObjectPage",
entitySet: "Bookings",
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
"authenticationMethod": "route",
"routes": [
{
"source": "^/alp/(.*)$",
"target": "/alp/$1",
"source": "^/analytics/(.*)$",
"target": "/analytics/$1",
"destination": "sflight-srv",
"authenticationType": "xsuaa",
"csrfProtection": false
Expand Down
File renamed without changes.
12 changes: 6 additions & 6 deletions mta-java.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ modules:
properties:
SPRING_PROFILES_ACTIVE: cloud
JBP_CONFIG_COMPONENTS: "jres: ['com.sap.xs.java.buildpack.jre.SAPMachineJRE']"
JBP_CONFIG_SAP_MACHINE_JRE: '{ use_offline_repository: false, version: 17.+ }'
JBP_CONFIG_SAP_MACHINE_JRE: '{ use_offline_repository: false, version: 17.+ }'
requires:
- name: sflight-db
- name: sflight-uaa
Expand Down Expand Up @@ -67,9 +67,9 @@ modules:
artifacts:
- travel-processor.zip
target-path: resources/
- name: sflight-app-travel-alp
- name: sflight-app-travel-analytics
artifacts:
- travel-alp.zip
- travel-analytics.zip
target-path: resources/

# ------------- APPLICATION: TRAVEL PROCESSOR ----------------
Expand All @@ -85,11 +85,11 @@ modules:
- npm run build
supported-platforms: []

# ------------- APPLICATION: TRAVEL ALP ----------------------
- name: sflight-app-travel-alp
# ------------- APPLICATION: TRAVEL ANALYTICS ----------------
- name: sflight-app-travel-analytics
# ------------------------------------------------------------
type: html5
path: app/travel_alp
path: app/travel_analytics
build-parameters:
build-result: dist
builder: custom
Expand Down
Loading

0 comments on commit be4a00a

Please sign in to comment.