Skip to content

Conversation

@toastercup
Copy link
Member

@toastercup toastercup commented Jul 8, 2017

Purpose:

This PR implements a very rudimentary RESTful API for Beta Cortex's ContentItems and ContentTypes. This is by no means future-proofed and will be scrapped in a couple quarters - but it accomplishes the goal of liberating our data. Lacking are easy ways for the consuming application to query FieldItem data, retrieve related ContentItems<->ContentItems, etc. I've purposefully commented out some lines of code as placeholders - these will be implemented in another PR I plan on pushing this weekend or next (implementing tenancy, some very basic filtering and finding, etc). This PR also implements a POC for the Employer Integration ContentType. This was put together a few weeks back to demonstrate how quickly we can spin up new ContentTypes as needs arise.

JIRA:

N/A

Steps to Take On Prod

N/A - eventually, we'll want to seed the Employer Integration ContentType, but since it's a POC and is not complete, we will hold off on doing so.

Changes:

  • Changes to setup

    • N/A
  • Architectural changes

    • N/A
  • Migrations

    • You must manually seed the new Employer Integration ContentType - it is not yet part of the reseed task, as it is still a POC. You can manually seed via this command: bundle exec rake employer:integration:seed
  • Library changes

    • N/A
  • Side effects

    • N/A

Screenshots

  • Before
    N/A

  • After
    N/A

QA Links:

http://web.cortex-1.development.c66.me/

How to Verify These Changes

Relevant PRs/Dependencies:

This is 1/2 of the PRs (the other will be opened in cortex-client shortly) needed to implement #512

Additional Information

N/A

@toastercup toastercup added this to the Custom Content milestone Jul 8, 2017
@toastercup toastercup changed the title Basic Beta Cortex API Basic Beta Cortex API + Employer Integration POC Jul 8, 2017
@arelia
Copy link
Contributor

arelia commented Jul 9, 2017

Creating a media item gives a NoMethodError

image

@toastercup toastercup force-pushed the content-items-endpoint branch from f43708d to 5091ea8 Compare July 10, 2017 00:35
@toastercup
Copy link
Member Author

toastercup commented Jul 10, 2017

@arelia this seems to be a bug already present in develop - we'll have to get a defect in to address it. Good catch!

@mrfantasticwonders
Copy link

I'm noticing the same error as Arelia when creating a media type. Not sure if this was addressed yet beyond the defect that will be created.

@toastercup
Copy link
Member Author

toastercup commented Jul 10, 2017

@mrfantasticwonders per my previous comment:

@arelia this seems to be a bug already present in develop - we'll have to get a defect in to address it. Good catch!

Copy link
Contributor

@MKwenhua MKwenhua left a comment

Choose a reason for hiding this comment

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

Looks good!

@toastercup toastercup merged commit 025958f into develop Jul 10, 2017
@toastercup toastercup deleted the content-items-endpoint branch July 10, 2017 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants