Skip to content

Commit

Permalink
fixed our api not using List and instead relying on Get to retire…
Browse files Browse the repository at this point in the history
  • Loading branch information
CommanderStorm committed Oct 3, 2023
1 parent 70958ab commit 02ce42c
Show file tree
Hide file tree
Showing 14 changed files with 975 additions and 979 deletions.
2 changes: 1 addition & 1 deletion client/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,5 @@ standard system TLS CA's configured to securely connect via `api-grpc.tum.app` t

Alternatively you can also test the API using [grpcurl](https://github.com/fullstorydev/grpcurl) (also located in the `testLiveApi.sh`):
```
grpcurl -protoset <(buf build -o -) -H "x-device-id:grpc-tests" api-grpc.tum.app:443 api.Campus/GetNewsSources
grpcurl -protoset <(buf build -o -) -H "x-device-id:grpc-tests" api-grpc.tum.app:443 api.Campus/ListNewsSources
```
2 changes: 1 addition & 1 deletion client/local/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ func queryDish(canteen string, dish string, c pb.CampusClient, ctx context.Conte
}

func queryCanteen(s string, c pb.CampusClient, ctx context.Context, imageShouldBeStored bool) {
res, err := c.GetCanteenRatings(ctx, &pb.GetCanteenRatingsRequest{
res, err := c.ListCanteenRatings(ctx, &pb.ListCanteenRatingsRequest{
CanteenId: s,
Limit: 3,
// From: timestamppb.New(time.Date(2022, 7, 8, 16, 0, 0, 0, time.Local)),
Expand Down
2 changes: 1 addition & 1 deletion client/testLiveApi.sh
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
#!/bin/bash

grpcurl -protoset <(buf build -o -) -plaintext -H "x-device-id:grpc-tests" api.tum.app:50052 api.Campus/GetNewsSources
grpcurl -protoset <(buf build -o -) -plaintext -H "x-device-id:grpc-tests" api.tum.app:50052 api.Campus/ListNewsSources
Loading

0 comments on commit 02ce42c

Please sign in to comment.