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

Allow FAQs to be shown on home/sponsor/registration pages #80

Merged
merged 1 commit into from
May 31, 2017

Conversation

epsy
Copy link
Collaborator

@epsy epsy commented May 24, 2017

@epsy epsy force-pushed the faqs-everywhere branch 2 times, most recently from 57611e7 to 18ff11a Compare May 25, 2017 05:49
@epsy
Copy link
Collaborator Author

epsy commented May 25, 2017

screenshot 2017-05-24 at 22 55 09

pybay/urls.py Outdated
from pybay.views import (
pybay_cfp_create, pybay_sponsors_list,
pybay_faq_index, pybay_speakers_list)
from pybay import views

Choose a reason for hiding this comment

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

This change seems more a matter of taste than what the PR is asking for and adds an extra namespace that is not needed. Lets not mix the two.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@mzdaniel
Copy link

Except for the comment, this looks like a fantastic job Yann! The PR is in staging now for UX review.

urlpatterns = [
url(r"^$", TemplateView.as_view(template_name="frontend/index.html"), name="home"),
url(r"^$", faq_view(template_name="frontend/index.html", faq_filter="show_on_home"), name="home"),
Copy link

Choose a reason for hiding this comment

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

Why are you calling this faq_view? isn't it home page?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It uses data from the FAQs. If the name is too confusing I could change it to something else

@PirosB3
Copy link

PirosB3 commented May 31, 2017

@mzdaniel merged when you are happy with this

@grace-law
Copy link

Yann, this is great stuff. I anticipate this feature will save me and other devs over 20 hours a week getting content on the website and editing.

Minor changes:

  1. Can you please move the "Frequently Asked Questions" text to center and make that Black (over-ride current CSS rules) for home, tickets and sponsors prospectus page.
  2. Can you please change the questions to the darker Blue and Bold (same size)

@mzdaniel
Copy link

I am merging this code now. Great work Yann!

@mzdaniel mzdaniel merged commit c5053b5 into pybay:master May 31, 2017
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.

4 participants