ESDet / esdportal_profile

drupal install profile for portal.excellentschoolsdetroit.org
0 stars 4 forks source link

Live Push from 10/14 Sprint/Crawl #44

Closed anderoy closed 8 years ago

anderoy commented 8 years ago

As we have discussed in a separate issue #39 we will accept an interim fix on the calculated fields. The following was the work I did that needs to be added to the live site. Check off boxes if it is included in the push. Thanks.

bnchdrff commented 8 years ago
e3_contact-teacher-field_e3_cohort
e3_contact-teacher-field_e3_status
e3_contact-teacher-field_training_attendance
e3_contact-teacher-field_training_hours_f1
e3_contact-teacher-field_training_hours_f2
e3_contact-teacher-field_training_hours_f3
e3_contact-teacher-field_training_hours_m1
e3_contact-teacher-field_training_hours_m2
e3_contact-teacher-field_training_hours_s1
e3_contact-teacher-field_training_hours_s2
e3_contact-teacher-field_training_hours_total
e3_contact-teacher-field_training_hours_w1
e3_contact-teacher-field_training_hours_w2 
bnchdrff commented 8 years ago

merging into the field-calculation branch & will push to live with those bugfixes after that's done

bnchdrff commented 8 years ago

merged in https://github.com/ESDet/esdportal_e3/commit/65095df1a557440f7e1aa71b1c8d94af8e81edaa

eric, i'm not sure what "Add 10x2 dimension “sub-pages’ with two attachment tables that contextually filter to the sub-page dimension." is referring to - is this a view contained in the feature already?

for " Change “Data Entry” role to “E3 Admin” and modify permissions only person currently on “Data Entry” is Bernita (who just so happens to be an E3 Admin). We don’t need the data entry role as it existed anymore." we'll need to make the user role changes manually when we push to live.

anderoy commented 8 years ago

The 10x2 pages are just different pages I added to the E3 Teacher Report view. If you look at the view ui page for that you will see what I mean. If you committed my changes to code, based on the changes I made to the views, it should be there.

anderoy commented 8 years ago

PC, PC Line, NC, NC Line (for example)

anderoy commented 8 years ago

I ended up needing to make a few "hotfixes" or minor adjustments to a few things on the live site in order to get things cleaned up a bit. Here are the changes: