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.

Sign up for GitHub

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

Jump to bottom

Extend POI model #726

Closed
svenseeberg opened this issue Mar 3, 2021 · 1 comment · Fixed by #1376
Closed

Extend POI model #726

svenseeberg opened this issue Mar 3, 2021 · 1 comment · Fixed by #1376
Assignees
@timobrembeck
Labels
🍼 good first issue Good for newcomers 💡 feature New feature or request ❗ prio: medium Should be scheduled in the forseeable future.
Milestone
Migration

Comments

@svenseeberg
Copy link
Member

svenseeberg commented Mar 3, 2021 •
edited by ulliholtgrave

Motivation

The map feature requires to have additional fields for our POI model.

Proposed Solution

The following (optional) attributes are necessary:

  • Website address (URL)
  • Contact e-mail address
  • Contact phone number
  • (Opening hours) TBD
  • Icon (Original)
  • Icon (Thumbnail_150x150)

This issue is deeply connected to #951 and could be solved together with it.
@svenseeberg svenseeberg added the 💡 feature New feature or request label Mar 3, 2021
@svenseeberg svenseeberg added this to the Additional Features milestone Mar 3, 2021
@timobrembeck
Copy link
Member

When implementing this, the POI-model parts of #695 and #522 could be included as well:

  • Add boolean field show_location_on_map
  • Add text field meta_description

@timobrembeck timobrembeck added 🍼 good first issue Good for newcomers ⁉️ prio: low Not urgent, can be resolved in the distant future. labels Mar 3, 2021
@timobrembeck timobrembeck changed the title Extend POI model Extend POI model May 7, 2021
@ulliholtgrave ulliholtgrave modified the milestones: Backlog, Version 1.2 Apr 27, 2022
@svenseeberg svenseeberg added ❗ prio: medium Should be scheduled in the forseeable future. and removed ⁉️ prio: low Not urgent, can be resolved in the distant future. labels Apr 27, 2022
@timobrembeck timobrembeck self-assigned this Apr 30, 2022
@timobrembeck timobrembeck mentioned this issue Apr 30, 2022
Merged
@timobrembeck timobrembeck mentioned this issue May 9, 2022
Merged
@timobrembeck timobrembeck mentioned this issue Mar 4, 2023
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timobrembeck timobrembeck

Labels
🍼 good first issue Good for newcomers 💡 feature New feature or request ❗ prio: medium Should be scheduled in the forseeable future.
Projects
None yet
Milestone
Migration
Development

Successfully merging a pull request may close this issue.

Add additional fields to location model digitalfabrik/integreat-cms
3 participants
@svenseeberg @ulliholtgrave @timobrembeck