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

Person Cemetery Location #68

Merged
merged 17 commits into from
May 3, 2021
Merged

Person Cemetery Location #68

merged 17 commits into from
May 3, 2021

Conversation

nafiesl
Copy link
Owner

@nafiesl nafiesl commented Apr 6, 2021

Description

Based on this feature proposal #63. This PR will add a cemetery location for a person who already passed away. This feature inspired by this Laravel Leaflet Example project. (Actually copy-paste the feature from there 😀)

Checklist

  • Add location_name, location_address, latitude, and longitude fields for the person's cemetery location.
  • Add a new table user_metadata to store addtitional user attribute (outside the users table).
  • Add getMetadata('some_key') method on the User model to get the current user's additional attribute from the user_metadata table.
  • Add a leaflet map to pin the person's cemetery location.
  • Show the cemetery location map on the user's profile page.
  • Add a config file leaflet.php to determine the default map center and zoom levels.
  • Add Show in Google Maps link.

Screenshots

The person's death edit page

screen_2021-04-06_003

The person's death section page

screen_2021-04-22_007

Here is a screencast record for this PR.

https://www.youtube.com/watch?v=AUcw6Ds9RH4

@nafiesl nafiesl mentioned this pull request Apr 6, 2021
@nafiesl nafiesl marked this pull request as ready for review May 3, 2021 15:03
@nafiesl nafiesl merged commit 6e18a0f into master May 3, 2021
@nafiesl nafiesl deleted the cemetery_location branch May 3, 2021 15:03
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.

1 participant