merge develop into master#82
Open
kushal2011 wants to merge 177 commits into
Open
Conversation
# Conflicts: # .gitignore
Edit the api config to use mlab database
- participant Detail
- code to show these icons on success/unsuccess of barcode scanning. -Added AttendeeStatusModel. -mentioned above jpegs icons on pubspec.yaml
# Conflicts: # app/raize/pubspec.lock # app/raize/pubspec.yaml
…from issue#6 and merged in DetailWidget.dart.
# Conflicts: # app/raize/lib/main.dart # app/raize/pubspec.lock # app/raize/pubspec.yaml
2)Connecting EventListWidget with EventDetailsWidget. 3)Removed linkingi of EventDetailsWidget from SignInWidget.
Ticket api improv
Use mongodb instead of csv file
-added mongodb to check schwag status and schwag given or not with validations
Use mongodb instead of csv file for swags api
-added mongodb condition fix
-fixed json message response on scanning renamed reason to message
-fixed swag model
Include a Pull Request template to help community folks contribute better.
Include an Issue template to help community folks contribute better.
code clean up and added comments for event list
SnehPandya18
reviewed
Oct 3, 2019
mistryhardik
reviewed
Oct 3, 2019
mistryhardik
left a comment
mistryhardik
left a comment
Member
There was a problem hiding this comment.
@SnehPandya18 You were supposed to delete the values for the keys, not the file itself.
Member
|
@kushal2011 What does Develop mean? This is not a valid PR title. Also there is just no description provided for all the changes we want to merge into master. Please use the following template to create the description and then we will review the PR once again: https://github.com/gdgbaroda/raize/blob/develop/PULL_REQUEST_TEMPLATE.md. @SnehPandya18 Thank you for the quick edit. |
Collaborator
Author
|
@mistryhardik done , changed the title and added description |
SnehPandya18
suggested changes
Oct 3, 2019
SnehPandya18
left a comment
SnehPandya18
left a comment
Contributor
There was a problem hiding this comment.
Proper pull request details and formatting requested
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
this pr contains the working app demo with following functionalities
-upcomming events are displayed to user
-button to scan qr code for registration and swag distribution is showed only to host in event details screen
-scaning the qr code will return a message for either success or failure