digitalfabrik / integreat-app

React JS and React Native App for Integreat
https://integreat.app
MIT License
45 stars 15 forks source link

IGAPP-1295: Automatically link existing POI on an event page upon creation #2223

Open jira-to-github-migrator[bot] opened 10 months ago

jira-to-github-migrator[bot] commented 10 months ago

Vijay Karthik - 16.5.2023, 13:22:48

Motivation

When I post an event at a location already stored in Integreat and use the map function, the location information in the event article does not link directly to the map entry.

Proposed Solution

Upon entering the location data for an event, the location is automatically linked to the already existing POI. The users can click on the link and view the location on the map.

User Story

As a municipality CMS user, I want the location information in the event article to be directly linked to the map entry, when the entry already exists, so that I need not enter any additional text and links in the editor form everytime.

Additional Context

Linked issues:

Depends on:

jira-to-github-migrator[bot] commented 10 months ago

Vijay Karthik - 16.5.2023, 13:31:46

for eg: https://integreat.app/regensburg/de/events/m26-kinderworkshop-kunstbetrieb-cham$2023-05-16

M26 is already a POI.

f1sh1918 - 17.5.2023, 14:18:33

added closed cms issue: https://github.com/digitalfabrik/integreat-cms/issues/2277

cms should deliver a poi link and an address

LeandraH - 17.5.2023, 17:30:59

Turns out CMS do deliver that information, it's just a little hard to find. In the event, we have a location.id, and in the location, we have an id at the top level which is not the one we need (it's from the translation of the POI), we need the location.id in here as well. We should be able to map those to each other.

Timo Ludwig - 17.5.2023, 18:09:55

Although the relationship is already contained in the data via the id, we'll add the link as well to avoid having to retrieve the whole location endpoint just to get the url:

https://github.com/digitalfabrik/integreat-cms/issues/2294

timobrembeck commented 8 months ago

The required change is now released in the production CMS: the events endpoint has the additional field location_url.

osmers commented 2 months ago

@steffenkleinle as far as I can see there is no prio on this issue - we have had a few municipalities in the last weeks ask about this (during Netzwerktreffen and two in the chat). I cannot estimate how difficult this issue is, but maybe we can plan it in for the nearer future? @dkehne since you are the product owner, maybe you can determine a time horizon :)

dkehne commented 2 months ago

end of 2024 - or 2025. we got a lot of issue which are more impactful.

LeandraH commented 2 weeks ago

We're getting the wrong URL here, opened a CMS issue: https://github.com/digitalfabrik/integreat-cms/issues/2852

LeandraH commented 6 days ago

The CMS issue has been merged, we can start working on this after the next CMS release 🥳