ScottStarkey, 10 days ago

@John_Livingston

Hi John - I would be happy to copy-edit for you. I have an English writing bachelor's degree...
... et je parle un peu français.

You did a fantastic job for a person who learned English as a second language. That being said, I would make the following changes:

"All room’s admins..." at several places in the document should be rooms' (with the apostrophe after the s).

[1/2]

John_Livingston, 10 days ago

@ScottStarkey

Hi Scott,

I see in your profile that you are developper (in addition to be a magician, which give us a second point in common). Maybe do you have a github account? In such case, you can propose a Pull Request by modifying directly this page:
https://github.com/JohnXLivingston/peertube-plugin-livechat/blob/main/support/documentation/content/en/documentation/user/streamers/tasks.md

Otherwise, i will apply your suggestions. Thanks!

ScottStarkey, 10 days ago

@John_Livingston

Hi John - I guess we are brothers from another mother. 🤓 🥰

I will check out your GitHub.
I am slowly migrating my own projects to @Codeberg

John_Livingston, 10 days ago

@ScottStarkey

If you prefer me to make the modifications, just tell me. It will be quick.

ScottStarkey, 10 days ago

@John_Livingston
"he/she gets instantly access"

You should probably say "they get instant access" or "they instantly get access". I would prefer the first option, though.

"You can also edit existing tasks list"
I would say "You can also edit existing task lists"

Also later you have "tasks lists" should probably be "task lists". Task is an adjective modifying lists, and thus singular.

[2/3]

John_Livingston, 10 days ago

@ScottStarkey

for "tasks lists" instead of "task lists", i just missed it... I knew it should be "task lists".

thanks

ScottStarkey, 10 days ago

[I will review and edit on GitHub !]

Merci et à bientôt

[3/3]

John_Livingston, 10 days ago

@ScottStarkey

i made the fix your suggested:
https://github.com/JohnXLivingston/peertube-plugin-livechat/commit/66c0d207dae7adc7c8e9dedf4ccf3447f358f305

John_Livingston, 10 days ago

@ScottStarkey
Just checked for "all room's admin": in this case, i mean "all admins of 1 room". So i guess "room's admin" is ok?

ScottStarkey, 10 days ago

@John_Livingston

Ah, yes. In that case it should be "every room's admins" instead of "all room's admins".

I made some other changes in in the pull request.

John_Livingston, 10 days ago

@ScottStarkey

oh ok, you were working on it.
Great!

ScottStarkey, 10 days ago

@John_Livingston

You may need to fix my "all rooms'" to "every room's".

John_Livingston, 10 days ago

@ScottStarkey
i will

John_Livingston, 10 days ago

@ScottStarkey
Great work! Thank you so much for your help :)