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.

Sign up for GitHub

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

Jump to bottom

Json Stringify #7

Closed
rittamdebnath opened this issue Jun 16, 2016 · 3 comments
Closed

Json Stringify #7

rittamdebnath opened this issue Jun 16, 2016 · 3 comments

Comments

@rittamdebnath
Copy link
Owner

screen shot 2016-06-16 at 8 23 00 am

Instead of JSON Stringify, can you make it display only the seat value and the seat letter like

You have selected 3 seats , 3B,3C,3D.
rittamdebnath referenced this issue Jun 16, 2016
@rittamdebnath
JSON converted to seats
c81ceba 
Still there is a mistake in the JSON object `sessionInfo.checkedSeats` variable, it doesn't contain seats that are selected in different lines such as ( 9G,10G,10H).
@rittamdebnath
Copy link
Owner Author

#7 Similarly 1G,1H,2H

@rittamdebnath
Copy link
Owner Author

from now on make all the changes in the ver1.1 the controller.js,service.js and module.js are inside scripts/quantity/ folder.

@jimut
Copy link
Contributor

jimut commented Jul 2, 2016 

currentSelectionSession.checkedSeats[rang][seatIndex] = seat;

This line was overwriting the seats of the same columns. Solved this one.
Creating a pull request in a moment.

@jimut jimut mentioned this issue Jul 2, 2016
Merged
rittamdebnath added a commit that referenced this issue Jul 2, 2016
@rittamdebnath
Merge pull request #10 from Jimut/ver1.1
663338b 
Fixed issues #6, #7 and #9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rittamdebnath @jimut