[CSE-69] Include asset_id on relevant tables - Githubissues

SafetyCulture / SafetyCulture-Power-Query-Connector

The SafetyCulture Power Query Connector provides a data connection for SafetyCulture in Power BI

MIT License

7 stars 2 forks source link

[CSE-69] Include asset_id on relevant tables #44

Closed BCook98 closed 1 year ago

BCook98 commented 1 year ago

Summary

Includes asset_id column on relevant tables.

Problem description

Pros/cons of approach implemented

Checklist

[ ] Is this PR a reasonable size?

Code Review Guidelines for Reviewers

Try to review in a timely manner. Opinions/nitpicks should not be blockers. Pair on a call for non-trivial feedback.
Overall design and approach should follow established patterns. Don't try to make the PR perfect.
Try to identify edge cases, race conditions, over-engineering, lack of test coverage and complexity.
If you don't feel qualified to review the code, pass it on to someone who is.