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.

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

Support Location #84

Closed
hacdias opened this issue Nov 4, 2021 · 4 comments · Fixed by #90
Closed

Support Location #84

hacdias opened this issue Nov 4, 2021 · 4 comments · Fixed by #90

Comments

@hacdias
Copy link
Contributor

hacdias commented Nov 4, 2021

Hey - it would be really nice if Micropublish supported adding location to posts similar to Quill. Unfortunately Quill only adds Location to notes, but it'd be great if that was an option for all post types. 😃

@barryf
Copy link
Owner

barryf commented Nov 4, 2021

Thanks for the issue, @hacdias! The hosted version at Micropublish.net has a default list for each post type with a fairly standard set of properties, but there are two methods you could use if you wanted to show the location fields:

  1. Clone the Micropublish repo and run your own instance. You can then customise the config/properties.json file and add location to other post types.
  2. Update your Micropub server to specify the properties you want. This is a proposed Micropub extension1 to return a JSON object with specific properties and required-properties when a client requests ?q=post-types. For example, this is what my server sends: https://github.com/barryf/vibrancy/blob/master/src/http/get-micropub/config/post-types.js

Footnotes

  1. https://github.com/indieweb/micropub-extensions/issues/33

@hacdias
Copy link
Contributor Author

hacdias commented Nov 5, 2021

Thanks for the reply @barryf!

  1. That could be an option. I'll try that out. Would it also have the same "Find" functionality that you can see on Check-in?
  2. I didn't know about that extension and it looks very cool!

@barryf barryf closed this as completed Nov 12, 2021
@hacdias
Copy link
Contributor Author

hacdias commented Nov 23, 2021

@barryf I added location to note via the method you linked, but it does not show up. I tried removing other fields and that worked. But adding location did not.

@barryf
Copy link
Owner

barryf commented Nov 23, 2021

@hacdias My mistake - it's been a while since I wrote that logic! The location fields are nested within the checkin property as an h-card. I'll re-open this issue as an enhancement.

@barryf barryf reopened this Nov 23, 2021
This was referenced Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants