Skip to content

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?


| 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.

4 participants