bcgov / BCP-CMS

BC Parks Content Management System

Apache License 2.0

3 stars 3 forks source link

Added confirmation popup to create advisory #60

Closed siva-ka closed 3 years ago

siva-ka commented 3 years ago

Description

This PR includes the following proposed change(s):

BC-119

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Does the change impact or break the Docker build?

[ ] Yes
[ ] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 716b2dd603a25d5bbd7e6e0cedd0b6a7acdf0050 into 54db8276a4ae6d218bbe6089feaf5652940405f7 - view on LGTM.com

new alerts:

2 for Unused variable, import, function or class