-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MDS-6207] Show PDF of permit amendment on Permit Conditions page + auto-focus on click #3351
Conversation
services/core-web/src/tests/components/mine/Permit/PreviewPermitAmendmentDocument.spec.tsx
Show resolved
Hide resolved
Other than some TS issues, looks very cool and works well! |
3054ce8
to
b4cd914
Compare
services/common/src/interfaces/permits/permitCondition.interface.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome!
Just noticed a duplicated interface and a suggestion on removing an unnecessary ||
block.
@matbusby-fw Fixed 👍 |
Quality Gate failed for 'bcgov-sonarcloud_mds_common'Failed conditions |
Quality Gate passed for 'bcgov-sonarcloud_mds_minespace-web'Issues Measures |
Quality Gate failed for 'bcgov-sonarcloud_mds_core-web'Failed conditions |
Objective
MDS-6207
With all the data about a permit condition now coming over from the permit service, we now have access to the bounding box of where it came from in the original PDF. This PR adds a side-by-side view of the conditions when you click "Open Permit in Document Viewer", where we also highlight the condition in the original PDF on click. Note: This is behind a feature flag, and just added so we can see how well it works/ if it's worth polishing.