Skip to content
Snippets Groups Projects
Commit 3e398975 authored by Martin Lowe's avatar Martin Lowe :flag_ca:
Browse files

Merge branch 'malowe/main/141' into 'main'

Add patch to fix limit use in loading GH install list

See merge request !157
parents c016af9e ee8ddf1b
No related branches found
No related tags found
1 merge request!157Add patch to fix limit use in loading GH install list
Pipeline #31764 passed
......@@ -148,7 +148,11 @@ public class DefaultGithubApplicationService implements GithubApplicationService
// build query to do fetch of records for currently active application
MultivaluedMap<String, String> params = new MultivaluedMapImpl<>();
params.add(GitEcaParameterNames.APPLICATION_ID_RAW, Integer.toString(config.github().appId()));
return dao.get(new RDBMSQuery<>(wrap, filters.get(GithubApplicationInstallation.class), params));
// we don't want to use the limit, as we want to pull all of the records we can find
RDBMSQuery<GithubApplicationInstallation> q = new RDBMSQuery<>(wrap, filters.get(GithubApplicationInstallation.class), params);
q.setUseLimit(false);
return dao.get(q);
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment