Skip to content

Conversation

@LeJeanbono
Copy link
Contributor

@LeJeanbono LeJeanbono commented Oct 4, 2025

Fixes #1127

@LeJeanbono LeJeanbono force-pushed the project-templates branch 3 times, most recently from 10c8512 to 9d79920 Compare October 4, 2025 13:04
@LeJeanbono
Copy link
Contributor Author

LeJeanbono commented Oct 4, 2025

Feel Free to make some feedback @jmini, fix #1127

@LeJeanbono LeJeanbono changed the title Add project Template list (#1127) Add project Template list #1127 Oct 4, 2025
* The token is revoked and a new one which will expire in one week is created to replace it.
* Only working with GitLab 16.0 and above.
*
*ProjectTemplate
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks a wired addition to the javadoc

List<ProjectTemplate> projectTemplates = unmarshalResourceList(ProjectTemplate.class, "project-templates.json");
assertTrue(compareJson(projectTemplates, "project-templates.json"));
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be possible to add a similar test for ProjectTemplateDetail.java

get(Response.Status.OK, queryParams, "projects", getProjectIdOrPath(projectIdOrPath), "iterations");
return (response.readEntity(new GenericType<List<Iteration>>() {}));
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Javadoc is missing here.

return (response.readEntity(new GenericType<List<ProjectTemplate>>() {}));
}

public ProjectTemplateDetail getProjectTemplate(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Javadoc is missing here.

@LeJeanbono
Copy link
Contributor Author

Thanks @jmini, should be ok now :)

Copy link
Collaborator

@jmini jmini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thank you!

@jmini jmini merged commit 5daafe9 into gitlab4j:main Oct 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing support for projects/:id/templates

2 participants