Render more properties from q=source #132

Closed
opened 2018-10-15 11:15:03 +02:00 by grantcodes · 7 comments
grantcodes commented 2018-10-15 11:15:03 +02:00 (Migrated from github.com)

Was just trying out the posts list query and although it grabbed my list of posts, it didn't handle formatting them very well. This may be a case of supporting displaying more property types, or there might be some mf2 to jf2 conversion you can do to improve the display of the posts

So far I have noticed the following properties are hidden:

  • summary
  • featured
  • like-of
  • bookmark-of
  • photo
  • category
  • published
  • visibility
  • post-status

Also since I return an object with html and value in the content property, that is not rendered correctly.

Expected behavior
I wouldn't expect indigenous to support rendering and editing all of these properties, although the most important ones personally would be post-status, photo, category, visibilty and fixing content. And on the post list if the post would end up being rendered as blank like in my screenshot, it should at least show the url.

I would also like to see that data in the edit screen, not a blank form. And I think the form should show all unrecognized properties properties (just as json is fine), so that you know you aren't going to lose any data. I was a bit worried about actually doing an update as I don't know if it is going to replace everything or just the new content I write.

Screenshots
screenshot_20181015-105751_1

Was just trying out the posts list query and although it grabbed my list of posts, it didn't handle formatting them very well. This may be a case of supporting displaying more property types, or there might be some mf2 to jf2 conversion you can do to improve the display of the posts So far I have noticed the following properties are hidden: - `summary` - `featured` - `like-of` - `bookmark-of` - `photo` - `category` - `published` - `visibility` - `post-status` Also since I return an object with `html` and `value` in the `content` property, that is not rendered correctly. **Expected behavior** I wouldn't expect indigenous to support rendering and editing all of these properties, although the most important ones personally would be `post-status`, `photo`, `category`, `visibilty` and fixing `content`. And on the post list if the post would end up being rendered as blank like in my screenshot, it should at least show the url. I would also like to see that data in the edit screen, not a blank form. And I think the form should show all unrecognized properties properties (just as json is fine), so that you know you aren't going to lose any data. I was a bit worried about actually doing an update as I don't know if it is going to replace everything or just the new content I write. **Screenshots** ![screenshot_20181015-105751_1](https://user-images.githubusercontent.com/3390268/46940624-73569e00-d069-11e8-9e1f-7c72c66c1019.jpg)
swentel commented 2018-10-15 11:33:02 +02:00 (Migrated from github.com)

I wonder if it's fine to ellipsize the content to say 2 lines or so ?

As for the current update screen: this is related to https://github.com/swentel/indigenous-android/issues/114 - I don't have support for that yet in the client, so clicking on update passes on the url and that's it.

When you would hit the send button on an update screen, only the properties which are not empty, except for the post status are send along in the payload. However, I agree, it's currently not optimal at all :) In the end, the goal is that the proper 'post' screen is selected, with all values pre-populated, so I need to do some post-type discovery first.

I'll be working on this towards Berlin, so I hope to get it finished by then!

I wonder if it's fine to ellipsize the content to say 2 lines or so ? As for the current update screen: this is related to https://github.com/swentel/indigenous-android/issues/114 - I don't have support for that yet in the client, so clicking on update passes on the url and that's it. When you would hit the send button on an update screen, only the properties which are not empty, except for the post status are send along in the payload. However, I agree, it's currently not optimal at all :) In the end, the goal is that the proper 'post' screen is selected, with all values pre-populated, so I need to do some post-type discovery first. I'll be working on this towards Berlin, so I hope to get it finished by then!
grantcodes commented 2018-10-15 11:37:48 +02:00 (Migrated from github.com)

Perfect, sounds like a good plan.

Feel free to let me know if you want some testing, I think I might be the only other android user that supports all these micropub query features at the moment :P

Perfect, sounds like a good plan. Feel free to let me know if you want some testing, I think I might be the only other android user that supports all these micropub query features at the moment :P
swentel commented 2018-11-02 09:17:55 +01:00 (Migrated from github.com)

There's a new release. It's rendering some more properties already, but it's not fully done yet, stay tuned :)

There's a new release. It's rendering some more properties already, but it's not fully done yet, stay tuned :)
dshanske commented 2019-01-01 02:04:30 +01:00 (Migrated from github.com)

I also have the content problem with returning an object @grantcodes mentioned

I also have the content problem with returning an object @grantcodes mentioned
swentel commented 2020-01-24 16:37:07 +01:00 (Migrated from github.com)

Working on this a bit, both on the Drupal side, and then here: the content object, is it inside an array also ? or not. Same question for the author basically.

Working on this a bit, both on the Drupal side, and then here: the content object, is it inside an array also ? or not. Same question for the author basically.
swentel commented 2020-04-18 11:45:33 +02:00 (Migrated from github.com)

Baby steps: the overview now handles content when the first key in the array is

  • a string
  • or an object with 'text' and 'html'. I render the text version at the moment as my current goal for this in indigenous is just to get a quick overview of the posts on my site.

Next step is update: getting the right post screen and calling q=source&url={url}

This fix will probably will come in a release in a few days already though.

Baby steps: the overview now handles content when the first key in the array is - a string - or an object with 'text' and 'html'. I render the text version at the moment as my current goal for this in indigenous is just to get a quick overview of the posts on my site. Next step is update: getting the right post screen and calling q=source&url={url} This fix will probably will come in a release in a few days already though.
swentel commented 2020-06-01 22:58:40 +02:00 (Migrated from github.com)

Data is fetched from the server now for an update. Closing this for now, it's fine for my use case at the moment. Other properties are way more complex, at that point, I have my content more easily updated in my cms ;)

Data is fetched from the server now for an update. Closing this for now, it's fine for my use case at the moment. Other properties are way more complex, at that point, I have my content more easily updated in my cms ;)
This discussion has been locked. Commenting is limited to contributors.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
inhji/indiepass-android#132
No description provided.