Skip to content

Conversation

@dreis2211
Copy link
Contributor

Hi,

this PR fixes a link in the docs, which leads to a 404 at the moment.
I also took the freedom to polish this a bit more and align this with other sections where we link to the application.properties Appendix.

Before

image

After

image

Cheers,
Christoph

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 13, 2022
@wilkinsona
Copy link
Member

Thanks for spotting this, @dreis2211. While the link's only broken in 3.0 snapshots, I think it'd be nice to link to a more specific part of the appending in all our maintained branches. We can then fix the broken link as part of merging that change forwards.

@wilkinsona wilkinsona added type: documentation A documentation update and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 13, 2022
@wilkinsona wilkinsona added this to the 2.5.x milestone Feb 13, 2022
@wilkinsona wilkinsona added the for: merge-with-amendments Needs some changes when we merge label Feb 13, 2022
@wilkinsona wilkinsona changed the title Fix link to Kafka properties in documentation Link directly to the Integration Properties section of the appendix appendix when cross-referencing Kafka properties Feb 13, 2022
@wilkinsona wilkinsona changed the title Link directly to the Integration Properties section of the appendix appendix when cross-referencing Kafka properties Link directly to the Integration Properties section of the appendix when cross-referencing Kafka properties Feb 13, 2022
@wilkinsona wilkinsona self-assigned this Feb 14, 2022
@wilkinsona wilkinsona removed the for: merge-with-amendments Needs some changes when we merge label Feb 14, 2022
@wilkinsona
Copy link
Member

Thanks again, @dreis2211.

@wilkinsona wilkinsona modified the milestones: 2.5.x, 2.5.10 Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: documentation A documentation update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants