Fix the current location error on iOS #249
Assignees
Labels
Comments
23mafi
added a commit
that referenced
this issue
Feb 15, 2022
* add navigation route to button * navigate to center of mass when clicking navigate button in detail view * use other navigation button, fix tests * delete puts and fix redeclaration of variable * rename css class, fix failures on dev * change icon of navigation button * Removed force_ssl which was part of a fix for #249 but lead to the login not working anymore * delete unnessary files * fix indoor map being built before outdoor map Co-authored-by: Marie Fischer <[email protected]> Co-authored-by: Judith <[email protected]> Co-authored-by: NikkelM <[email protected]> Co-authored-by: Paul Sieben <[email protected]>
NikkelM
added a commit
that referenced
this issue
Feb 17, 2022
* First working version * Work done * Closes #185 * Track all changes * converting search results to geojson Co-authored-by: Benjamin Frost <[email protected]> * adds markers for search results Co-authored-by: Benjamin Frost <[email protected]> * added HPI geocoder * added reverse geolocation * doesnt show students for not logged in users as search results * Map 19/navigation start (#188) * implements navigation route, that takes two points and starts navigation. * navigation route uses the current position as starting point * adds test * linting * changed "context" in test * refactor: auto format scss * fix: detail and search page list height * test: add to_string tests * feat: add course search * fix: add informations to people factory * test: add and refactor search tests * run cucumber tests workflow on push (#246) * persist query * use local storage * empty * fix test * add test * add option for hamburger menu * add id * Fix routing popup showing when room popup is opened * First test for room popup * refactors first test for room popup * fix first test for room popup * fix room tests * fix room tests * made case sensitive tests * wip * fix room tests * wip * case insensitve search (maybe) ^^ * wip * case insensitive ? * rubocop friendly * refactoring, bugfixes and adding normal geocoding * fit map bounds to markers Co-authored-by: Laura Meister <[email protected]> * fit map bounds to search result markers * fix selection of one room * Update features/scenarios/search/show_search_results.feature Co-authored-by: Cedric Lorenz <[email protected]> * 111-related-matches (#247) * Return related searchable records * Sort more results Make related results less prioritized * Fix and refactor tests * Add tests and fix sorting * Use to_s instead of to_string * Update app/controllers/search_controller.rb Co-authored-by: elenagensch <[email protected]> * Update app/controllers/search_controller.rb Co-authored-by: elenagensch <[email protected]> * Update app/controllers/search_controller.rb Co-authored-by: elenagensch <[email protected]> * Update app/controllers/search_controller.rb Co-authored-by: elenagensch <[email protected]> * Update features/scenarios/search/search_for_related.feature Co-authored-by: elenagensch <[email protected]> * Correct keyword in cucumber test * Change comment in search controller * Adjust comment again * Linting issues * Make tests green * Refactor feature tests * Format code and change .rubocop.yml * Fix tests and tag priority sorting * Update searchable_record.rb * Fix tests from merge Co-authored-by: sm1lla <[email protected]> Co-authored-by: Smilla Fox <[email protected]> Co-authored-by: elenagensch <[email protected]> * updates people model and changes prohibited keys to person view * updates migration * fix people show * fix people search test * update db * debug deployment * Redesign search page * fix search tests * removed token comment * Center the search vertically * inital webmanifest * rename variable and switch from layergroup to layer * correct indentation * start implementing Ajax approach * try to send remote call * fix search index * fix empty string bug * add new feature * add feature test for search result display * format js * Set force_ssl=true (#268) (#269) Co-authored-by: NikkelM <[email protected]> Co-authored-by: Nick Bessin <[email protected]> Co-authored-by: NikkelM <[email protected]> * fix missing import * fix bottombar options redirect * implement redirects in options page * implement pure ajax and change to only html for form * adopt search results partial to master state * 211-update chair ui (#254) * update chair ui * implemented requested changes * make placeholder chair image square Co-authored-by: Margarete <[email protected]> * create detail page for courses (#255) Co-authored-by: Margarete <[email protected]> * feat: added css for circular pictures * poi ui without tests * write tests for poi detail page * refactor: rubocop corrections * fix: variable assignment * implement redirects in not signed in options page * adds options page to navbar * changes options icon in navbar * add suggestion when there are no search results (#263) * add suggestion when there are no search results * do not show suggestions on first visit * update lockfiles * adopt new changes to search results on master * fix tests and start to adopt for map search * Closes 275 * fix: test poi name * add live search on map * add map marker refresh * make similar results only dependent on more results * fix: linting issues * Switch search bar positon on focus * fix: course picture * Fix zoom on 'Show on map' click * Fix popup not removing * fix linter issues * adopt search result structure of dev * add error message and make nice * Interface to select floor (#232) *added title and red box * multiple floor interfaces * fix(map): no overlaping layers when no room selected * Start working on search page * added loading search page according to entered query * Fix merge issues * 200 navigation button (#265) * add navigation route to button * navigate to center of mass when clicking navigate button in detail view * use other navigation button, fix tests * delete puts and fix redeclaration of variable * rename css class, fix failures on dev * change icon of navigation button * Removed force_ssl which was part of a fix for #249 but lead to the login not working anymore * delete unnessary files * fix indoor map being built before outdoor map Co-authored-by: Marie Fischer <[email protected]> Co-authored-by: Judith <[email protected]> Co-authored-by: NikkelM <[email protected]> Co-authored-by: Paul Sieben <[email protected]> Co-authored-by: tfiedlerdev <[email protected]> Co-authored-by: LeonHermann322 <[email protected]> Co-authored-by: Benjamin Frost <[email protected]> Co-authored-by: Tobias Sträubig <[email protected]> Co-authored-by: treyfel <[email protected]> Co-authored-by: rothaarlappen <[email protected]> Co-authored-by: Lukas Wenner <[email protected]> Co-authored-by: cdfhalle <[email protected]> Co-authored-by: elenagensch <[email protected]> Co-authored-by: Benjamin Frost <[email protected]> Co-authored-by: Tom Richter <[email protected]> Co-authored-by: MartinPreiss <[email protected]> Co-authored-by: Leon Hermann <[email protected]> Co-authored-by: DevSchmidtchen <[email protected]> Co-authored-by: Laura Meister <[email protected]> Co-authored-by: MartinPreiss <[email protected]> Co-authored-by: Cedric Lorenz <[email protected]> Co-authored-by: Abdullatif Ghajar <[email protected]> Co-authored-by: sm1lla <[email protected]> Co-authored-by: Smilla Fox <[email protected]> Co-authored-by: Kirill Postnov <[email protected]> Co-authored-by: Tobias Sträubig <[email protected]> Co-authored-by: Paul Sieben <[email protected]> Co-authored-by: Lucas Liebe <[email protected]> Co-authored-by: Franz Sauerwald <[email protected]> Co-authored-by: Nick Bessin <[email protected]> Co-authored-by: lucasliebe <[email protected]> Co-authored-by: richardschiemenz <[email protected]> Co-authored-by: Margarete <[email protected]> Co-authored-by: cdfhalle <[email protected]> Co-authored-by: Nikola Genchev <[email protected]> Co-authored-by: Kirill <[email protected]> Co-authored-by: 23mafi <[email protected]> Co-authored-by: Marie Fischer <[email protected]> Co-authored-by: Judith <[email protected]> Co-authored-by: Margarete01 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
(I am not sure how many people have the same problem I have, maybe I am just incompetent.)
Using the deployed version of the navigation on Safari on my iPhone, I can not access my current location, even though the permissions allowed it and Chrome did it just fine. I get the error
Geolocation error: User denied location. If this is indeed an error, it should be fixed, especially because the map will typically be used mainly on mobile where the share of people with iOS and Safari as their main browser is significant. Also check, if it works on mobile with Android, although chrome will be used there much more often.The text was updated successfully, but these errors were encountered: