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

WFPREV-100/101 SMK initialization & map interactions for project list #437

Open
wants to merge 32 commits into
base: main
Choose a base branch
from

Conversation

ssylver93
Copy link
Collaborator

@ssylver93 ssylver93 commented Jan 22, 2025

  • Update sonarscan.yml to use npm to fix yarn dependency errors
  • Initialize SMK and set baseMap/activeMap
  • Use leaflet for pins/clustering
  • Mock polygons - these will be replaced by spatial data from activities/projects endpoints
  • Project list/pin/polygon interaction

Pins/cluster layer:
image
mocked polygons:
image
Pin change/polygons become darker when pin is clicked or project is selected on map:
image

@ssylver93 ssylver93 requested review from yzlucas and dhlevi January 23, 2025 01:26
rm -rf node_modules
yarn cache clean
npm cache clean --force
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why are we doing this? do you got yarn issue on sonar test build??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes yarn was having issues finding lealfet dependencies for some reason. I changed to npm as I knew it was working for the other workflows, and didn't want to spend anymore time debugging.

Copy link
Collaborator

@yzlucas yzlucas left a comment

Choose a reason for hiding this comment

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

Looks good! Just a suggestion on a method for better readability,
Cant really tell the changes in Angular.json but I assume if it build and pass the unit tests we are fine.

@ssylver93 ssylver93 requested a review from yzlucas January 23, 2025 19:30
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