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

Events #24

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

Events #24

wants to merge 4 commits into from

Conversation

vinaymundada27
Copy link
Member

I have modified the events page as per Mayank Kedia said.
I guess the page is almost done!

@abhijithanilkumar
Copy link
Member

@vinaymundada27 Currently, the events page shows all the events including details. I don't think this page is of any use. When we click on any event in the calendar, it should take us to a page with the details of that particular event. So, I think we need a page like news where all the events are listed down (or we can chuck it since we have the calendar showing all the events). What do you guys think about this? @rohitvarkey @97yogitha

screenshot from 2016-03-14 21 07 20

@vinaymundada27 Also, do a rebase and move the urls and views you added to the website app. We have defined all other views and urls in website.

@rohitvarkey
Copy link
Contributor

@abhijithanilkumar Sounds good.

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.

3 participants