Skip to content

Conversation

@felipewind
Copy link
Contributor

A small correction on the toadmin description in the user manual documentation.

This was talked on #237

Local test:

image

From

This allows you to add fields before an adminstrative message before it is sent out.

To

This allows you to add fields to an administrative message before it is sent out.

@chrjohn
Copy link
Member

chrjohn commented Jan 16, 2021

Hi @felipewind , just stumbled over this... Could you do me a favour and also correct the following while you're at it...

<dd> This callback provides you with a peak at the administrative messages

This should be peek instead of peak.

Thanks in advance. :)

@felipewind
Copy link
Contributor Author

@chrjohn I did the change you request, from peak to peek.

I'm not sure if I did correctly the git commits. If I messed up the git log just say and I'll correct it.

thanks

@chrjohn chrjohn changed the title Corrects the toadmin description in the user manual documentation Corrects the toAdmin description in the user manual documentation Jan 17, 2021
@chrjohn chrjohn added this to the QFJ 2.2.1 milestone Jan 17, 2021
@chrjohn
Copy link
Member

chrjohn commented Jan 17, 2021

LGTM, thanks. 👍

@chrjohn chrjohn merged commit 09d6104 into quickfix-j:master Jan 17, 2021
@felipewind felipewind deleted the correction-usermanual-toadmin branch May 5, 2021 21:35
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