Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Mar 9, 2021. It is now read-only.

Single upcoming event to be shown #124

Closed
wants to merge 2 commits into from

Conversation

akshatm147
Copy link

#118 I changed the loadlimit size for the upcoming events to be 1.

@M-ZubairAhmed @pbteja1998 Please review.

--

Checklist for this PR

  • [ x] I have read contributing guidelines.
  • [ x] I executed yarn lint locally to format my code.
  • [ x] I added revelant Tests to verify my code.
  • [ x] All tests yarn test ran successfully.
  • [ x] Pull request is towards develop branch.
  • [ x] I have tested responsiveness of the UI by checking on devices using ScreenFly
  • [ x] Added myself to contributors table
  • [ x] Documentation
  • [ x] Ready to be merged

@welcome
Copy link

welcome bot commented Dec 13, 2017

Thanks so much for opening your first PR here!

@coderplex-bot
Copy link
Collaborator

Deploy preview ready!

Built with commit 7c3906b

https://deploy-preview-124--coderplex.netlify.com

Copy link

@M-ZubairAhmed M-ZubairAhmed left a comment

Choose a reason for hiding this comment

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

You should try to see how we are displaying each event card, and try to use that in index.js page/ landing page

@@ -30,7 +30,7 @@ export default class Events extends React.Component {
pastEvents: [],
pastEventsLoadLimit: 2,
futureEvents: [],
futureEventsLoadLimit: 2,
futureEventsLoadLimit: 1,

Choose a reason for hiding this comment

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

We arent looking at this to change.

@akshatm147
Copy link
Author

@M-ZubairAhmed I do not ReactJS so I am not able to get how to use that in the index.js file. Can you tell me what I should do.

@M-ZubairAhmed
Copy link

M-ZubairAhmed commented Dec 13, 2017

@chotuaashiq its inspiring that you would want to contribute, i would suggest you go through this ReactJS and look for components section. We have prepared a curated list of all react best learning resource. Also there is a webcast today where we are going to explain our code base. I would like to invite you for that and you can also ask lots of questions live.

@M-ZubairAhmed
Copy link

ping!

@vinaypuppal
Copy link
Contributor

Closing this for #133

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants