Comments

coltrane’s picture

Status: Active » Needs review

Will review this weekend. Thanks Sumit!

coltrane’s picture

Status: Needs review » Needs work

Need to remove the email address from api/speakers, cannot expose email addresses publicly like that. I will remove it before committing.

Since we have Core Conversations and Day Stage can we add those to the mobile schedule? They are currently included on http://denver2012.drupal.org/program/schedule

What do you think about stripping HTML tags from node body of the Views?

coltrane’s picture

Since my concerns in #2 can be handeled with config if you agree I will commit and we can modify from there.

sumitk’s picture

Coltrane please do that. Also please get me an admin account on drupalcon site, I need that to manage app data.

sumitk’s picture

I will see what I can do about code conversations. HTML tags stripping and clean ups are taking place on app.

Once we have live data ill send app for trials!

sumitk’s picture

StatusFileSize
new5.95 KB
new4.46 KB

Update
- Remove 6.x form .info file
- Remove email field from speakers

sumitk’s picture

StatusFileSize
new6.2 KB

Fixed items per page for schedule view

coltrane’s picture

cod_mobile_speakers

Are you displaying Core Conversations and Day Stage items on the schedule? I propose including them in this feed and exposing the content type field.

Are you using the "speaker" field of user rows? I recommend dropping it since name and uid are singular and for session with multiple speakers the session appears again in the JSON data.

     "user" : {
        "session_title" : "Tools, Tips, & Techniques - Project Management Using Hybrid Agile Development",
        "speaker" : "MatthewS, StaceyH",
        "name" : "MatthewS",
        "picture" : "http://denver.dev/sites/default/files/styles/thumbnail/public/pictures/picture-599-1312955141.jpg",
        "uid" : "599"
      }

We now have "Full name", "Company or organization" and "Job title" profile data from drupal.org so we can include that information in the API if you want to have the apps read it. I'll have it in the JSON feed.

cod_mobile_schedule

Additionally propose including Core Conversations and Day Stage items in this View.

I'm going to update the room field to use the formatter that includes sponsor name so we stay consistent when referring to rooms. "Four Kitchens room" is more memorable than "Room MHB 1A". The field label will stay the same.

cod_mobile_bofs

Not sure how you're handling node updates, but I think this View (and the other node ones as well) should include the nid to support object updates and avoid a duplicate BoF if someone changes the title.

BoF node type will have a taxonomy field per work in #1447442: Separate finding a BoF and scheduling a BoF so that can be included in the View.

coltrane’s picture

Status: Needs work » Active

COD Mobile App is now live on denver2012.drupal.org with the changes I mentioned in #8

Thanks for getting this going Sumit! Looking forward to getting access to testflightapp to see this.

The BoF API will go live the first week of March. When BoF submission is turned on for the site.

Another question, how often are you polling these pages for updates? Pending results of #1449460: Test performance and caches of site I may set up some Views time-based caches so want to be aware of your usage.

(leaving this issue open for any further features or change requests)

sumitk’s picture

As http://drupal.org/node/1293870 is fixed now - can we please enable bof mobile feature on live site. I need it urgently for app submissions. Thanks

coltrane’s picture

This is enabled http://denver2012.drupal.org/api/bofs but since BoFs aren't available yet there is nothing there. I suggest you do not wait for BoFs to start appearing before submitting.

sumitk’s picture

Thanks! it works now.

sumitk’s picture

Status: Active » Closed (fixed)
sumitk’s picture

Issue summary: View changes

Updated issue summary.