-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add Clustering to Location View Map #62
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
brebory
approved these changes
Aug 10, 2022
Closed
josephmaxwellhyland
added a commit
that referenced
this pull request
Oct 12, 2022
* chore: add support for nx workspaces * add root package.json * use add-nx-to-monorepo to add initial nx config * add info about nx workspaces to README.md * chore: simplify import path in passport.js * chore: add support for babel and jest * test: add initial tests * front-end * update App.test.js to use react-test-renderer instead of react-dom * address warnings from tests * Landing.js - change testAuth propType to func * Navbar.js - use "static" for AppBar position prop instead of "inherit" * back-end * add submitTable.test.js with basic validation tests * all * add eof newlines to some files that were missing them * chore: add GitHub Actions workflow for unit tests * front-end: add github-actions reporter to config * back-end: add github-actions reporter to config * Add Bugs for Interns (#13) * use specific db name for mongo * spelling mistakes * Just changed it to "logged" in * changed logn to login * changed "bellow" to "below" * Go to Dashboard * updated login ui using mui components * updated register ui with mui components * revert unnecessary package changes * updated just the model, not the routes/api. * update location types * slight tweaks * based it off the table schema * update comment schema with foreign keys * rename pt 1 * rename pt 2 * fix comment convention * added new field and changed the route/api * added user ID * submit table testing * separate page for the map * prototype with google maps api * rename view map container * WIP table popup * use real data * back-end: add bounded check * turn off tests for now * cleanup * organizational changes * abstract out TableData component for ViewTable * add constants for latitude and longitude geojson index * fix axios caching * fix get tables within bounds * refactor location table search * handle interactive map changes * Update title and logo (#48) * Added the Comment, Picture, and Table changed * Updated the Title and Logo of the UCT Locator * Update Title and Logo * Update Title and Logo * Update FavIcon Logo * Update index.html file * Delete .DS_Store * Delete .DS_Store * Delete .DS_Store * Deleted Comment and Picture files * created a new file modelled after the table schema to make one for th… (#32) * created a new file modelled after the table schema to make one for the pictures * update picture schema * update picture schema * Remove unused ref * update picture schema * Remove unused ref * Re-do this Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> * changed tables.js to protect it from rejecting any tables that didn't have an enum based field filled out by adding if statements. Also created a form for logged in users to submit tables * formatting updates * submit table basic updates * whitespace fix * more whitespace dumb stuff * Add Selection Map to Submit Table Page (#59) * add TableMap ref to set zoom and center * just move location centering to TableMap component * updates * update to use centered marker to select locations * add map selection for submit table * ws * diff update * add nav drawer (#60) * Add Clustering to Location View Map (#62) * get marker clustering working basically * fix warnings, simplify code * simplify * get marker clustering working basically * fix warnings, simplify code * simplify * fix some merge issues * Autocomplete Library * small changes * feature: add breadcrumbs for navigation * Add Breadcrumbs component using MUI Breadcrumbs * Add Breadcrumbs as an always-matched route so every page renders them * Add Link component to make it easier to use react-router-dom Links wrapped in MUI Links * Add Containers to all pages to constrain to md maxWidth * Run prettier formatting on every changed file * chore: fix formatting in Link.js * chore: add support for migrate-mongo closes #34 * Fixed the autocomplete zoom and search * Add Developer Setup Guide (#88) Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> * add gitbook+startup learning (#89) Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> * update home link * added username field to user schema and register (#54) * added username field to user schema and register * special character guidlines, check name is in use * Update register.js * fix user existence validation * fix some whitespace Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> * 46 password requirements (#78) * used regex to update username requirements * used regex to create stronger password requirement Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> * Table data refactor (#92) * refactor table data * fix table data merging Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> Co-authored-by: Brendon Roberto <brendon.roberto@hyland.com> Co-authored-by: Joseph Maxwell (Hyland) <106829895+josephmaxwellhyland@users.noreply.github.com> Co-authored-by: Nihal Patil <nahalpattal@gmail.com> Co-authored-by: Aakarsh <aakarsh0607@gmail.com> Co-authored-by: Bella Baglier <isabellabaglier@gmail.com> Co-authored-by: Neel Nayar <neelnayar436@gmail.com> Co-authored-by: Joseph Maxwell <joseph.maxwell@hyland.com> Co-authored-by: neelnayar23 <61635741+neelnayar23@users.noreply.github.com> Co-authored-by: aakarshgarg23 <103973650+aakarshgarg23@users.noreply.github.com>
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When zooming out, markers that are close together will be grouped into a cluster. This also updates the location data retrieval - it simply grabs all of the locations at once, up front.