Skip to content

Commit

Permalink
fix(ui): missing manifests and logs (argoproj#9231)
Browse files Browse the repository at this point in the history
Signed-off-by: Michael Crenshaw <michael@crenshaw.dev>
  • Loading branch information
crenshaw-dev committed Apr 29, 2022
1 parent 1533698 commit 14e379a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions ui/src/app/shared/services/applications-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ export class ApplicationsService {
resourceName: resource.name,
version: resource.version,
kind: resource.kind,
group: resource.group
group: resource.group || '' // The group query param must be present even if empty.
})
.then(res => res.body as {manifest: string})
.then(res => JSON.parse(res.manifest) as models.State);
Expand Down Expand Up @@ -355,9 +355,7 @@ export class ApplicationsService {
const search = new URLSearchParams();
search.set('container', containerName);
search.set('namespace', namespace);
if (follow) {
search.set('follow', follow.toString());
}
search.set('follow', follow.toString());
if (podName) {
search.set('podName', podName);
} else {
Expand All @@ -377,6 +375,8 @@ export class ApplicationsService {
if (previous) {
search.set('previous', previous.toString());
}
// The API requires that this field be set to a non-empty string.
search.set('sinceSeconds', '0');
return search;
}

Expand Down

0 comments on commit 14e379a

Please sign in to comment.