API Documentation for /notifications/read is incomplete

[SabreCat]

The API documentation for "Mark multiple notifications as read" is inaccurate.

There is a required parameter in the request body, notificationIds, specifying what notifications to mark as read, which doesn't show up in apidoc as yet. We don't currently have a blanket API route to dismiss all notifs without listing them.

[shivamAmrutia]

This is my first issue and i have no clue how to get started, can you please tell me how can i help ?

[TUdarbe]

Hi! I would like to pick up and work on this issue.

[KipronohVincent]

Hello I'm requesting for guidance on how to tackle this issue

[marnamrs]

Hi! I have looked into controller files and saw the missing @apiParam, I can pick this up and add the annotation.

[KipronohVincent]

sure

…

On Wed, Feb 22, 2023 at 12:28 PM marnamrs ***@***.***> wrote: Hi! I have looked into controller files and saw the missing @apiParam, I can pick this up and add the annotation. — Reply to this email directly, view it on GitHub <#14484 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AVJAGT2QZJ2KZWSO66ITHXDWYXL3NANCNFSM6AAAAAAUQXLGCA> . You are receiving this because you commented.Message ID: ***@***.***>

[CuriousMagpie]

Sure thing, @marnamrs! Thanks!

[apoorvapendse]

is this issue resolved?

[apoorvapendse]

Do you want it to look like this?
@SabreCat @CuriousMagpie

[apoorvapendse]

@SabreCat @CuriousMagpie can i work on this?

[CuriousMagpie]

Sure thing, @apoorvapendse! Thanks!

[apoorvapendse]

@CuriousMagpie Thanks for assigning, do you also need the same change for marking multiple notifications as seen?

[CiceroLino]

is this issue resolved? if not, can I work on this?

[apoorvapendse]

@CiceroLino I have created a pull request for the same, waiting for it to be reviewed.

[ducknows]

does anyone got an update on this issue if got reviewed or not?

[apoorvapendse]

@ducknows nope.

[nadasuhailAyesh12]

Can I work on this issue, could you assign me please

[anishlukk123]

@SabreCat can you review the pr associated with this