diff --git a/src/main/java/com/openelements/issues/ApiEndpoint.java b/src/main/java/com/openelements/issues/ApiEndpoint.java index 89c59cc..f1f2b37 100644 --- a/src/main/java/com/openelements/issues/ApiEndpoint.java +++ b/src/main/java/com/openelements/issues/ApiEndpoint.java @@ -26,73 +26,6 @@ public ApiEndpoint(@NonNull final GitHubCache issueCache) { this.issueCache = Objects.requireNonNull(issueCache, "issueCache must not be null"); } - @Deprecated(forRemoval = true) - private record OldIssueResponse(@NonNull String title, @NonNull String link, @NonNull String org, @NonNull String repo, @NonNull String imageUrl, @NonNull String identifier, boolean isAssigned, boolean isClosed, @NonNull List labels, @NonNull List languageTags) { - } - - /** - * @deprecated Use {@link #getIssues(Boolean, Boolean, Set, Set, Set)} instead - * @return Set of good first issues - */ - @Deprecated(forRemoval = true) - @GetMapping("/api/hacktoberfest-issues") - public Set getHacktoberfestIssuesOld() { - log.warn("DEPRECATED API CALLED: Getting Hacktoberfest issues"); - return issueCache.getIssues(LabelConstants.HACKTOBERFEST_LABEL).stream() - .map(issue -> new OldIssueResponse(issue.title(), issue.link(), issue.repository().org(), issue.repository().name(), issue.repository().imageUrl(), issue.identifier(), issue.isAssigned(), issue.isClosed(), issue.labels(), issue.repository().languages())) - .collect(Collectors.toUnmodifiableSet()); - } - - /** - * @deprecated Use {@link #getContributors()} instead - * @return - */ - @Deprecated(forRemoval = true) - @GetMapping("/api/contributors") - public Set getContributorsOld() { - log.warn("DEPRECATED API CALLED: Getting Contributors"); - return getContributors(); - } - - /** - * @deprecated Use {@link #getIssues(Boolean, Boolean, Set, Set, Set)} instead - * @return Set of good first issues - */ - @Deprecated(forRemoval = true) - @GetMapping("/api/good-first-issues") - public Set getGoodFirstIssuesOld() { - log.info("Getting good first issues"); - return issueCache.getIssues(LabelConstants.GOOD_FIRST_ISSUE_LABEL).stream() - .map(issue -> new OldIssueResponse(issue.title(), issue.link(), issue.repository().org(), issue.repository().name(), issue.repository().imageUrl(), issue.identifier(), issue.isAssigned(), issue.isClosed(), issue.labels(), issue.repository().languages())) - .collect(Collectors.toUnmodifiableSet()); - } - - /** - * @deprecated Use {@link #getIssues(Boolean, Boolean, Set, Set, Set)} instead - * @return Set of good first issues - */ - @Deprecated(forRemoval = true) - @GetMapping("/api/good-first-issue-candidates") - public Set getGoodFirstIssuesCandidatesOld() { - log.info("Getting good first issue candidates"); - return issueCache.getIssues(LabelConstants.GOOD_FIRST_ISSUE_CANDIDATE_LABEL).stream() - .map(issue -> new OldIssueResponse(issue.title(), issue.link(), issue.repository().org(), issue.repository().name(), issue.repository().imageUrl(), issue.identifier(), issue.isAssigned(), issue.isClosed(), issue.labels(), issue.repository().languages())) - .collect(Collectors.toUnmodifiableSet()); - } - - /** - * @deprecated Use {@link #getIssues(Boolean, Boolean, Set, Set, Set)} instead - * @return Set of good first issues - */ - @Deprecated(forRemoval = true) - @GetMapping("/api/help-wanted-issues") - public Set getHelpWantedIssuesOld() { - log.info("Getting help wanted issues"); - return issueCache.getIssues(LabelConstants.HELP_WANTED_LABEL).stream() - .map(issue -> new OldIssueResponse(issue.title(), issue.link(), issue.repository().org(), issue.repository().name(), issue.repository().imageUrl(), issue.identifier(), issue.isAssigned(), issue.isClosed(), issue.labels(), issue.repository().languages())) - .collect(Collectors.toUnmodifiableSet()); - } - @GetMapping("/api/v2/contributors") public Set getContributors() { log.info("Getting contributors"); @@ -100,7 +33,11 @@ public Set getContributors() { } @GetMapping("/api/v2/issues") - public Set getIssues(@PathVariable(required = false) Boolean isAssigned, @PathVariable(required = false) Boolean isClosed, @PathVariable(required = false) Set filteredLabels, @PathVariable(required = false) Set excludedLabels, @PathVariable(required = false) Set filteredLanguages) { + public Set getIssues(@PathVariable(name = "isAssigned", required = false) Boolean isAssigned, + @PathVariable(name = "isClosed",required = false) Boolean isClosed, + @PathVariable(name = "filteredLabels",required = false) Set filteredLabels, + @PathVariable(name = "excludedLabels",required = false) Set excludedLabels, + @PathVariable(name = "filteredLanguages",required = false) Set filteredLanguages) { log.info("Getting good first issues"); return issueCache.getAllIssues().stream() .filter(issue -> isAssigned == null || issue.isAssigned() == isAssigned) diff --git a/src/main/resources/static/contributors.html b/src/main/resources/static/contributors.html index ff36366..8e1fcb3 100644 --- a/src/main/resources/static/contributors.html +++ b/src/main/resources/static/contributors.html @@ -27,7 +27,7 @@

Contributors