Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update optaplanner spring boot starter url #26816

Closed
wants to merge 2 commits into from

Conversation

ge0ffrey
Copy link
Contributor

@ge0ffrey ge0ffrey commented Jun 8, 2021

Previous URL is 404 now.
I also added the quickstart link, FWIW.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 8, 2021
Copy link
Member

@snicoll snicoll left a comment

Choose a reason for hiding this comment

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

Thanks for reporting the 404. I've added a comment, can you please have a look to it?

@@ -182,7 +182,7 @@ do as they were designed before this was clarified.
| https://github.com/okta/okta-spring-boot

| https://www.optaplanner.org/[OptaPlanner]
| https://github.com/kiegroup/optaplanner/tree/master/optaplanner-integration/optaplanner-spring-boot-starter
| https://github.com/kiegroup/optaplanner/tree/master/optaplanner-spring-integration/optaplanner-spring-boot-starter (https://github.com/kiegroup/optaplanner-quickstarts/tree/stable/spring-boot-school-timetabling[quickstart example])
Copy link
Member

Choose a reason for hiding this comment

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

Please revert the quickstart link. None of the other entries have such a link and I am not keen to introduce that. You can make sure the quickstart guide is prominent in whatever link you provide here.

Copy link
Contributor Author

@ge0ffrey ge0ffrey Jun 10, 2021

Choose a reason for hiding this comment

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

No problem, I see this issue was already fixed, so no need for me to adjust this myself. Thank you for merging.

@snicoll snicoll added the status: waiting-for-feedback We need additional information before we can continue label Jun 8, 2021
@philwebb philwebb added type: documentation A documentation update for: merge-with-amendments Needs some changes when we merge and removed status: waiting-for-feedback We need additional information before we can continue status: waiting-for-triage An issue we've not yet triaged labels Jun 8, 2021
@philwebb philwebb modified the milestones: 2.4.x, 2.5.x Jun 8, 2021
snicoll pushed a commit that referenced this pull request Jun 10, 2021
snicoll added a commit that referenced this pull request Jun 10, 2021
@snicoll snicoll closed this in bbdfcdc Jun 10, 2021
@snicoll snicoll modified the milestones: 2.5.x, 2.5.1 Jun 10, 2021
@snicoll snicoll self-assigned this Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
for: merge-with-amendments Needs some changes when we merge type: documentation A documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants