Issue description
When we initially started DNS Records, we set the permissions to IS_STAFF.
The Dns Records view should be allowed by/restricted to domain managers and staff managing the domain.
It should be IS_DOMAIN_MANAGER and IS_STAFF_MANAGING_DOMAIN
In a previous PR, IS_DOMAIN_MANAGER was added, but an added test was not easily fixed, so punting that to this issue since it was out of scope for the other one.
This will require testing in both as an analyst and an application user.
Acceptance criteria
- Only domain managers and staff managing a domain can view dns records
- This test is "unskipped" and passing:
registrar.tests.test_views_domain.TestDomainPermissions.test_domain_pages_blocked_for_on_hold_and_deleted_for_dns_records
Additional context
Updates will be in the DnsRecordsView and the permissions.py
Additionally, tests and test data will need updating for tests to pass.
Links to other issues
No response
Issue description
When we initially started DNS Records, we set the permissions to IS_STAFF.
The Dns Records view should be allowed by/restricted to domain managers and staff managing the domain.
It should be IS_DOMAIN_MANAGER and IS_STAFF_MANAGING_DOMAIN
In a previous PR,
IS_DOMAIN_MANAGERwas added, but an added test was not easily fixed, so punting that to this issue since it was out of scope for the other one.This will require testing in both as an analyst and an application user.
Acceptance criteria
registrar.tests.test_views_domain.TestDomainPermissions.test_domain_pages_blocked_for_on_hold_and_deleted_for_dns_recordsAdditional context
Updates will be in the DnsRecordsView and the permissions.py
Additionally, tests and test data will need updating for tests to pass.
Links to other issues
No response