Skip to content

Adds API support to all pages#99

Merged
matt-bernhardt merged 1 commit into89-rebuild-apifrom
matt/feature/other-pages
Sep 20, 2016
Merged

Adds API support to all pages#99
matt-bernhardt merged 1 commit into89-rebuild-apifrom
matt/feature/other-pages

Conversation

@matt-bernhardt
Copy link
Copy Markdown
Member

@matt-bernhardt matt-bernhardt commented Sep 13, 2016

This adds markup to relevant pages so that the card loading plugin can configure itself correctly. It ends with the plugin logging the loaded content to the browser console. Future work will extend this code to include pagination control, as well as rendering the content as HTML.

Affected pages include:

  • Homepage (/)
  • News page (/news)
  • Events page (/events)
  • Single page (related posts)
  • Category pages (/category/*)
  • Bibliotech page (/bibliotech-index/)
  • Search page (/?s=)

@frrrances - we've discussed this at intervals, but do you want to look over the code before I merge?

@matt-bernhardt matt-bernhardt self-assigned this Sep 13, 2016
@matt-bernhardt matt-bernhardt force-pushed the matt/feature/other-pages branch 8 times, most recently from 4690b3e to 73c827d Compare September 20, 2016 16:21
This PR ends with logging loaded posts to the browser console. Pagination, and card rendering, are next.

Signed-off-by: Matt Bernhardt <mjbernha@mit.edu>
@matt-bernhardt matt-bernhardt force-pushed the matt/feature/other-pages branch from 73c827d to d6e7589 Compare September 20, 2016 17:24
@matt-bernhardt matt-bernhardt changed the title [WIP] Adds API support to all pages Adds API support to all pages Sep 20, 2016
Copy link
Copy Markdown
Contributor

@frrrances frrrances left a comment

Choose a reason for hiding this comment

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

Looks solid to me!

@matt-bernhardt matt-bernhardt merged commit 84bf196 into 89-rebuild-api Sep 20, 2016
@matt-bernhardt matt-bernhardt deleted the matt/feature/other-pages branch September 20, 2016 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants