readthedocs / ext-theme

Read the Docs drop in replacement site templates
2 stars 2 forks source link

Notifications: show attached to `User`, `Project` and `Organization` #261

Closed humitos closed 10 months ago

humitos commented 10 months ago

Render notifications attached to these objects. This is the first pass of this work and there are some things we need to make decisions and improve.

Note we are rendering these notifications in the template for now, but in the future they will be rendered using the APIv3: #259

Screenshots

User

Screenshot_2024-01-11_11-01-33

Project

Screenshot_2024-01-11_11-01-40

Organization

Screenshot_2024-01-11_11-01-16

Closes #260

humitos commented 10 months ago

I'm showing all the notifications attached to Project and Organization. However, the endpoint we are creating will only show these notifications to admins/owners. @agjohnson should I do the same here by returning a more specific queryset via the get_context() in the view?

humitos commented 10 months ago

I would put the project/organization notifications at the top of the content page, above the project/organization header.

Done in 84073ef

It looks like this now:

Screenshot_2024-01-16_09-36-11