Skip to content
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

Map adding points #5

Merged
merged 31 commits into from
Jul 2, 2024
Merged

Map adding points #5

merged 31 commits into from
Jul 2, 2024

Conversation

RyanBirtch-aot
Copy link
Owner

Description

Added map component, added drawing functionality, submissions and point retrieval works.

Types of changes

New feature (non-breaking change which adds functionality)

@@ -31,6 +31,8 @@
},
"dependencies": {
"@bcgov/bc-sans": "^1.0.1",
"@bcgov/smk": "^1.2.1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove unused packages smk, smk/cli

@@ -1,5 +1,6 @@
<!DOCTYPE html>
<html lang="en">
<script src="../node_modules/@bcgov/smk/dist/smk.js"></script>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove as not using smk

@@ -47,8 +47,14 @@
"components"
],
"dependencies": {
"@bcgov/smk": "^1.2.1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update dependecies with used library

}

attach(element) {
console.log('Attaching component to element');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove the console logs from the codes

@abhilash-aot abhilash-aot merged commit 5dd1e7d into feat-map Jul 2, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants