Skip to content

Conversation

@matrei
Copy link

@matrei matrei commented Oct 23, 2025

@bkoehm Will this make the docs clearer on request map usage?

@bkoehm
Copy link
Owner

bkoehm commented Oct 24, 2025

Thank you @matrei. I can merge this into my branch, but before I do, I have a question. I did not originally notice that Section 3.7 had a listing for the request map domain class. I think instead of me duplicating it in Section 4.6, I should have just linked to the class in Section 3.7. Do you agree? If so, I can make that change.

@matrei
Copy link
Author

matrei commented Oct 24, 2025

Thank you @matrei. I can merge this into my branch, but before I do, I have a question. I did not originally notice that Section 3.7 had a listing for the request map domain class. I think instead of me duplicating it in Section 4.6, I should have just linked to the class in Section 3.7. Do you agree? If so, I can make that change.

@bkoehm Sounds good! Go ahead!

@bkoehm bkoehm merged commit b6b79ea into bkoehm:bkoehm.7.0.x.1179 Oct 24, 2025
@matrei matrei deleted the docs-requestmap-suggestion branch October 24, 2025 15:58
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.

2 participants