Skip to content

Conversation

@TylerAdamMartinez
Copy link
Contributor

Why

This PR addresses the following problem/context:

  • Use bullet points here

How

Implementation summary - the following was changed/added/removed:

  • Use bullet points here

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Use bullet points here

@TylerAdamMartinez
Copy link
Contributor Author

Captura de pantalla 2026-01-23 a la(s) 18 08 45

@jirhiker
Copy link
Member

looks good. One minor comment, creating new NMA Weather Photos records should not be allowed, i.e. remove/disable the "New NMA Weather Photos" button. FYI this should be the case for all NMA_ tables; all NMA_ tables should be read-only

Copy link
Member

@jirhiker jirhiker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make all NMA_ admin views read-only

@chasetmartin
Copy link
Collaborator

chasetmartin commented Jan 24, 2026

@TylerAdamMartinez Here is the pattern to make the view read only

def can_create(self, request: Request) -> bool:
return False
def can_edit(self, request: Request) -> bool:
return False
def can_delete(self, request: Request) -> bool:
return False

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.

4 participants