Skip to content

Commit

Permalink
Merge pull request #216 from Screenly/fix/delete_app_list
Browse files Browse the repository at this point in the history
Fix: don't show deleted app in the app list.
  • Loading branch information
korvyashka authored Aug 27, 2024
2 parents 60c53a0 + ab08819 commit cc41386
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/commands/edge_app/app.rs
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ impl EdgeAppCommand {
pub fn list(&self) -> Result<EdgeApps, CommandError> {
Ok(EdgeApps::new(commands::get(
&self.authentication,
"v4/edge-apps?select=id,name",
"v4/edge-apps?select=id,name&deleted=eq.false",
)?))
}

Expand Down Expand Up @@ -1094,6 +1094,8 @@ mod tests {
let edge_apps_mock = mock_server.mock(|when, then| {
when.method(GET)
.path("/v4/edge-apps")
.query_param("select", "id,name")
.query_param("deleted", "eq.false")
.header("Authorization", "Token token")
.header(
"user-agent",
Expand Down

0 comments on commit cc41386

Please sign in to comment.