search

JackHaK / TMF

0 stars 0 forks source link

Re-Map Administrate elements #27

Open PeteHaK opened 5 years ago

PeteHaK commented 5 years ago

Now Alex has remapped the data elements we can take this opportunity to re-map the JSON elements returned by the integration tier. Do you want to tidy this up and reflect Alex's elements: Course Overview Testimonial Course Content Course Benefits PreRequisites Assessment Tutor Profile Partner Course Length Next Steps

seanthepottingshed commented 5 years ago

@PeteHaK @joshgosselin

Thanks for the above however the mapping seems to be misaligned somewhere:

See below:

tutorProfile needs to return an array of tutorProfiles comprising of the following:

partners needs to return an array of partners comprising of the following:

JackHaK commented 5 years ago

Sean,

646 isn’t in the list of courses Adam has set up – use 125 which is.

The tutor profile data you have asked for doesn’t exist in Administrate – if we need photos then the integration tier can cater for it but we should discuss the model we want to use.

Partners is a text field in Adams mapping – course-text-8 – so I’m not sure how we can return an array – again lets discuss.

Apologies for the categories data – it doesn’t get reinflated when we rebuild from scratch and we had to rebuild to cater for the new data – I’ll inflate it now.

Pete

From: Sean Davidson notifications@github.com Sent: 09 January 2019 11:38 To: JackHaK/TMF TMF@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: Re: [JackHaK/TMF] Re-Map Administrate elements (#27)

@PeteHaK https://github.com/PeteHaK @joshgosselin https://github.com/joshgosselin

Thanks for the above however the mapping seems to be misaligned somewhere:

tutorProfile needs to return an array of tutorProfiles comprising of the following:

partners needs to return an array of partners comprising of the following:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, https://github.com/JackHaK/TMF/issues/27#issuecomment-452666853 view it on GitHub, or https://github.com/notifications/unsubscribe-auth/AmronM--_NLwTTBuj40D7QQoDSgxigXtks5vBdSrgaJpZM4ZpxS- mute the thread. https://github.com/notifications/beacon/AmronJK27snW3BSW52a9IxqE3_PpJ57yks5vBdSrgaJpZM4ZpxS-.gif