nextcloud / nextcloud/maps

Geo property of a contact is ignored by Maps after update from 18.0.2 to 18.0.4 [on Yunohost]

Open
#401 5 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
598
Forks
113
PR merge metrics
No merged PRs in 30d

Description

That issue might come from somewhere at Yunohost (server, app…), but I cannot tell. I thought you could simply check on your side and tell them. Corresponding Yunohost issue report : nextcloud_ynh/#309 . See also: #16.

Steps & results:

  1. Install a fresh up-to-date Nextcloud app (18.0.4) on an up-to-date Yunohost server (tested: same date of this issue report).
  2. In NC-Maps, see:
    • All contacts with a geo property (and no adr) are now missing, although they were properly displayed using 18.0.2.

In addition, behaviour is consistent (and that is a good thing) whatever means you try to use to set or reset a geo property:

  1. Remove and set back geo property on a contact (using NC-Contacts).
  2. or Create a fresh new contact, and add a geo property but no adr (using NC-Contacts).
  3. or Import a contact with a geo property but no adr (using NC-Contacts).
  4. In NC-Maps, see:
    • Same results as before: missing, but worked in previous version.

Can you reproduce that, possibly on a regular NC instance?

Notes:

  • The address book I used had 951 real-life contacts, 45 of which had a geo property (and no adr). All these 45 cards do not appear anymore in Maps, but others are displayed normally.
  • I did not try to add or modify a contact by external Carddav client, because no one I have supports geo property.

By the way, geo support is a great unique feature of NC (too bad it vanished on my instance). I suppose it could be bound to an adr property as a param for more consistency, but anyway things are already very useful.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the missing geo-only contacts in NC-Maps 18.0.4 on Yunohost, comparing the result with 18.0.2. Read the linked Yunohost issue nextcloud_ynh/#309 and related issue #16; the work is done when contacts with a geo property but no adr appear in Maps again.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.