Skip to content

Commit

Permalink
bug: [DRON-433]: fix new build query logic (#424)
Browse files Browse the repository at this point in the history
  • Loading branch information
Dan Wilson authored Oct 24, 2022
1 parent a3a5a98 commit ca55d7d
Showing 1 changed file with 10 additions and 8 deletions.
18 changes: 10 additions & 8 deletions src/pages/repo/repo.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -96,16 +96,18 @@ const Repo = ({ user }) => {
const handleNewBuildClick = toggleModal;

const handleNewBuildSubmit = useCallback(async (values) => {
let endpoint = `/api/repos/${namespace}/${name}/builds`;
const queryParams = new URLSearchParams();
if (values.target) {
endpoint = `${endpoint}?branch=${values.target}`;
if (values.commit) {
endpoint = `${endpoint}&commit=${values.commit}`;
}
if (values.parameters?.length) {
endpoint = `${endpoint}&${values.parameters.map((param) => `${param.key}=${param.value}`).join('&')}`;
}
queryParams.set('branch', values.target);
}
if (values.commit) {
queryParams.set('commit', values.commit);
}
if (values.parameters?.length) {
values.parameters.forEach((param) => { queryParams.set(param.key, param.value); });
}
const queryString = queryParams.toString();
const endpoint = `/api/repos/${namespace}/${name}/builds${queryString.length ? `?${queryString}` : ''}`;
try {
const newBuild = await axiosWrapper(endpoint, {
method: 'POST',
Expand Down

0 comments on commit ca55d7d

Please sign in to comment.