-
Notifications
You must be signed in to change notification settings - Fork 175
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
Updated SparkSummit17 sample #242
Conversation
@Jigar3 give a deployment link of the sample generated through web app generator. |
Here is the deployment link: |
Hi @Jigar3! Looks like your PR has some conflicts. 😟 |
This reverts commit 404ce17.
Hi @Jigar3! Looks like your PR has some conflicts. 😟 |
|
||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Give a line space here.And similar spaces are required at various places.
@Jigar3 also the sample generated is not generated correctly. |
@Jigar3 change the state key to accepted instead of draft in sessions json that would generate speakers .Rest I am debugging the remaining fields. |
It seems like this PR has not been completed. I am closing this due to inactivity. Please open a new PR taking care of requested changes, if you are still interested to participate. Thank you. |
#241 Updated the SparkSummit Sample according to the new API