Skip to content

UP-4797 Adds lifecycle to json layout feed#815

Merged
vertein merged 1 commit intouPortal-Project:masterfrom
vertein:UP-4794-AddsLifecycle
Jan 30, 2017
Merged

UP-4797 Adds lifecycle to json layout feed#815
vertein merged 1 commit intouPortal-Project:masterfrom
vertein:UP-4794-AddsLifecycle

Conversation

@vertein
Copy link
Contributor

@vertein vertein commented Jan 27, 2017

Checklist
  • the [individual contributor license agreement] is signed
  • commit message follows [commit guidelines]
  • tests are included
Description of change

Adds lifecycle to JSON layout feed. Useful for those not using the rendering pipeline. For example, AngularJS would like to render a 'Maintenance mode' widget for items in maintenance mode.

Example:
"content": [
{
"_objectType": "portlet",
"url": "/uPortal/api/v4-3/portlet/what-is-uportal.html",
"iconUrl": "/uPortal/media/skins/icons/mobile/feedback.png",
"ID": "u26l1n1110",
"chanID": "53",
"description": "Description of uPortal.",
"fragment": "0",
"precedence": "80.0",
"fname": "what-is-uportal",
"locale": "en_US",
"name": "Welcome to uPortal",
"timeout": "10000",
"title": "Welcome to uPortal",
"typeID": "3",
"deleteAllowed": "false",
"moveAllowed": "false",
"windowState": "normal",
"portletMode": "view",
"portletName": "cms",

"lifecycleState": "PUBLISHED",

"webAppName": "/SimpleContentPortlet",
"parameters": {
"disableDynamicTitle": "true",
"configurable": "true",
"iconUrl": "/ResourceServingWebapp/rs/tango/0.8.90/32x32/mimetypes/text-html.png",
"mobileIconUrl": "/uPortal/media/skins/icons/mobile/feedback.png"
}
},

@coveralls
Copy link

Coverage Status

Coverage increased (+0.009%) to 23.033% when pulling d620391 on vertein:UP-4794-AddsLifecycle into 6ad27b7 on Jasig:master.

Copy link
Member

@ChristianMurphy ChristianMurphy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@drewwills
Copy link
Contributor

Great idea!

@drewwills
Copy link
Contributor

I assume this JSON is available via a RESTful endpoint. Is that endpoint versioned? Is this a change that would make us consider incriminating the version of this feed? (I'm not saying it's necessary, but it's probably something we should review & consider.)

@vertein
Copy link
Contributor Author

vertein commented Jan 27, 2017

it's available via the 4.3 layout endpoint (https://github.com/Jasig/uPortal/blob/master/uportal-war/src/main/java/org/apereo/portal/rest/layout/LayoutJsonV43RenderingController.java#L70). I haven't been following the versioning discussion that closely. Have we been making a new controller every time we add some data? Do we up the version on a release? What kind of work are we looking for?

@vertein vertein force-pushed the UP-4794-AddsLifecycle branch from d620391 to d984a09 Compare January 30, 2017 15:45
@vertein vertein merged commit 00695b0 into uPortal-Project:master Jan 30, 2017
@ChristianMurphy ChristianMurphy added this to the 5.0.0 milestone Apr 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants