Project widget V1

[caleybrock]

Recent projects and static new project list.

Moved ContentContainer to make it more generalized.

[caleybrock]

I added this to a page by adding the following:

#projects-widget to a haml file.

To a js file:

  $.ajax({
    method: 'GET',
    url: `/v3/channels`,
    dataType: 'json'
  }).done(projectLists => {
    const widget = document.getElementById('projects-widget');
    projectLists.sort((a, b) => {
      if (a.updatedAt < b.updatedAt) {
        return 1;
      } else {
        return -1;
      }
    });
    projectLists = projectLists.filter(project => {
      return !project.hidden;
    });
    projectLists = projectLists.slice(0,4);
    let convertedProjectList = [];
    for (let i = 0; i < projectLists.length; i++){
      let convertedProject = {
        name: projectLists[i].name,
        channel: projectLists[i].id,
        thumbnailUrl: projectLists[i].thumbnailUrl,
        type: projectLists[i].projectType,
        updatedAt: projectLists[i].updatedAt
      };
      convertedProjectList.push(convertedProject);
    }
    ReactDOM.render(
      <Provider store={getStore()}>
        <ProjectWidget projectList={convertedProjectList}/>
      </Provider>,
      widget);
  });
});

^This is not a super clean way to pass data in, just a way to prove we have the data setup in a way that will work with our components.

This work item didn't include adding this to pages yet, but the student page and course overview pages might be ones to come.

Component is 970px wide, our new width going forward, so it should work well on other pages, but was tested on the projects page.

[Bjvanminnen]

I would prefer just inlining renderButtons here

[Bjvanminnen]

I would again recommend inlining renderProjectCardList.

[Bjvanminnen]

nit: would prefer to break up these long lines (here and 263)

[Erin007]

nit: space before />

[Erin007]

Should we use the styling for the linkToViewAll in ContentContatiner here, so it's consistent?

[caleybrock]

This was unused CSS, removed.

[Erin007]

I'm confused about why some of these const are CAPS_SNAKE and some are lowerCamel. Should they be consistent?

[caleybrock]

I was also confused. Updating to all lowerCamel.

[Erin007]

Thanks for doing all of the work to move ContentContainer out of studioHomepages!

Is the goal to be able to just have something like: <ProjectWidget/> on the student homepage and /courses? Data for that will be the next step?

[caleybrock]

Data exists but will probably be different because not all pages are set up the same. I'll talk to poorva/tanya about if we want the V1 version on any pages, before implementing V2.

[caleybrock]

@Bjvanminnen updated based on review.

[Bjvanminnen]

nit: could be const