Skip to content

Event form implementation#224

Open
mashhoodr wants to merge 7 commits intorecurship:masterfrom
Sarmad93:event-form-signup
Open

Event form implementation#224
mashhoodr wants to merge 7 commits intorecurship:masterfrom
Sarmad93:event-form-signup

Conversation

@mashhoodr
Copy link
Copy Markdown
Member

No description provided.

path="/events/:event_id"
render={routeProps => (
<div>
{props.userState.token ? (
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@Sarmad93 we can access the event detail page without logging in. But when we click the button we need to redirect to the login page if we are not logged in

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Alright got it! will fix this one later tonight.

@mashhoodr
Copy link
Copy Markdown
Member Author

@saadullahaleem @Sarmad93 is having some issue submitting to the endpoint for forms submissions, its returning 500. Can you please have a quick look on that?

@Sarmad93
Copy link
Copy Markdown

@mashhoodr i think this one is ready for the final review.

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