Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Presentations page without database #52

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Presentations page without database #52

wants to merge 3 commits into from

Conversation

stevelaskaridis
Copy link
Contributor

Created a 3rd page in the webpage where the presentations available from past meetups are presented to the user.

More specifically:

  • Added the page, creating a new view (app/views/presentations/index.html.erb)
  • Changed the layout view to provide access to the new view
  • Minor addition in custom.css
  • Created a controller for the new view (app/controllers/presentations_controller.rb)
    • support for sorting
  • Added the relevant route
  • Created model Presentation class (app/models/presentation.rb)

Concerning pull request #50, the changes that were discussed were applied.

Screenshot preview:

knowledge-base-page

@stevelaskaridis
Copy link
Contributor Author

@petros, @xarisd, your feedback would be greatly appreciated ...

@xarisd
Copy link
Member

xarisd commented Feb 18, 2015

Guys I was busy with doctors and a funeral. I won't be available till next week.
@stevelaskaridis sorry for not replying sooner...

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