Skip to content

[MDS-6975] Give data team read only (Select) access to required public tables in Postgres - #3965

Open
alazar-aot wants to merge 1 commit into
developfrom
MDS-6975-Give-Data-Team-Access-to-Core-DB-Tables
Open

[MDS-6975] Give data team read only (Select) access to required public tables in Postgres#3965
alazar-aot wants to merge 1 commit into
developfrom
MDS-6975-Give-Data-Team-Access-to-Core-DB-Tables

Conversation

@alazar-aot

Copy link
Copy Markdown
Collaborator

Objective

MDS-

Why are you making this change? Provide a short explanation and/or screenshots

The data team informed us that they require SELECT (read only) access to the below public MDS tables:

Table
public.minespace_user_document_xref
public.notice_of_work_tier
public.major_mine_application_document_subtype
public.ams_final_application_document_xref
public.ams_final_application_version
public.minespace_user_version
public.minespace_user_request
public.minespace_user_role_xref
public.now_application_tier
public.email_tracking
public.permit_condition_tag_xref
public.standard_permit_condition_tag_xref
public.minespace_user_role
public.ams_final_application
public.now_application_tier_version
public.ams_final_application_document_type
public.permit_condition_tag

This PR adds that access to the user specified by the data team

…the mds_data_analytics user/role, to allow the data team to read from those tables
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@alazar-aot alazar-aot added the 👍 Ready for review Pull request has been double checked by the author and is ready for comments and feedback. label Aug 7, 2026

@matbusby-fw matbusby-fw left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good. The only possible issue I can see would be if we were to rename/drop any of these tables down the line (very unlikely), it might cause a headache for whichever dev took that on. But I think the risk is quite minimal for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👍 Ready for review Pull request has been double checked by the author and is ready for comments and feedback.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants