-
Notifications
You must be signed in to change notification settings - Fork 1
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
Export to XML/JSON formats #10
Comments
@Mikey1848 Your email said you would have this finished by Sunday and posted. We can't move forward until you do some work man. Can we shoot for at least 1 commit for this sprint? Baby steps. |
I was having some issues with my code and then ended up just procrastinating it if I'm being honest when I sent you that email. I'll update my code now to what has been pushed to github and try and fix it.
…________________________________
From: Taylor Flatt <[email protected]>
Sent: Monday, April 3, 2017 10:07:39 AM
To: SIU-CS/Flowchart-Creator-production
Cc: Michael Easton; Mention
Subject: Re: [SIU-CS/Flowchart-Creator-production] Export to XML/JSON formats (#10)
@Mikey1848<https://github.com/Mikey1848> Your email said you would have this finished by Sunday and posted. We can't move forward until you do some work man. Can we shoot for at least 1 commit for this sprint? Baby steps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#10 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AVFNCQw_9iNd2jqZmvrrmGWpcvn27tuBks5rsQs7gaJpZM4L8ndn>.
|
@Mikey1848 What was your error exactly? |
I don't know if I had the right idea but I was just trying to follow the format you used for the fromxml. I just committed my code to github under my branch. The error I am getting is foreach statement cannot operate on variables of type '?' because '?' does not contain a public definition for 'GetEnumerator'.
…________________________________
From: Taylor Flatt <[email protected]>
Sent: Monday, April 3, 2017 11:18:54 AM
To: SIU-CS/Flowchart-Creator-production
Cc: Michael Easton; Mention
Subject: Re: [SIU-CS/Flowchart-Creator-production] Export to XML/JSON formats (#10)
@Mikey1848<https://github.com/Mikey1848> What was your error exactly?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#10 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AVFNCWMBh6a4sD8kQDCb5Q6eVKjVVxAwks5rsRvugaJpZM4L8ndn>.
|
@Mikey1848 I don't see any commits under your branch or any others. What do you think the error means? |
Ok the code finally committed to my branch the error occurs at line 130 for selectSteps |
@Mikey1848 What do you think the error means? And you have 3 nested for-loops. Is there really no better way to handle that? |
Yeah looking at it now I should probably just rewrite this. |
@Mikey1848 That not withstanding, do you have any idea as to what the error means? |
I believe its because selectSteps is initialized as not having a type and I attempt to iterate over it as a collection. |
@Mikey1848 So how would you solve this? I mean my code that you copied it from isn't necessarily the code you should be using in this situation. Did you try debugging to see the state values? |
@Mikey1848 Also, you need to look up LINQ queries. You used the LINQ query that I made and they do have types. Var doesn't imply typelessness, it means that it is implicitly typed. Everything has a type in C#...that is a foundational element of the language. For LINQ queries, they use a thing called lazy loading. Google that and you'll hopefully understand a bit more. |
Alright thanks a lot for the help. |
I'm a complete idiot there was a period after app.Value for selectSteps I removed it and it fixed the error. |
@Mikey1848 Have you determined the saving/naming schema as well? This is all stuff we have had to wait on until this is done. Think you will have it done by 5pm or something? |
I have not. I can try and have it done by 5pm. |
@Mikey1848 You've given me literally 24 hours to do the vast portion of my contribution to this sprint man. @regexpressyourself has put a considerable amount of effort into his part and we aren't able to nicely connect it with the backend. We have to do better....be better. An employer won't be forgiving. Procrastination doesn't fly especially when other people depend on you. Need to respect the time of other people in your group. Keep us in the loop with where you get. |
@Mikey1848 I can empathize. But at the end of the day, as an adult, you have responsibilities. Even if that means letting us know that you're indisposed or not understanding things. We can work with that. Otherwise I really don't know what to do or say man. Are you going to be on campus today? We should probably discuss this off github. |
I'm done with classes for the day but I can meetup with you on campus. |
@Mikey1848 I don't mean to pile on you here, but here goes. I respect that you had some stuff going on, but you signed up for this class under the agreement that you can show up and get stuff done. Everyone has extra forces at play in their lives, but using that as an excuse to not follow through on your responsibilities won't fly in the real world and it doesn't fly with me. Like @taylorflatt said, your part of this project was the bridge between my work and Taylor's. Because you haven't come through for us, we've been forced to write around your work to get some basic functionality implemented. (See #59) Lying about progress is never ok. I can't trust you with any responsibilities moving forward when you openly and repeatedly lie to me. I've put in a ton of effort to pull you along, and gotten nothing in return. I don't know what else to say. It's on you to prove me wrong in sprint 3, but I'm done putting in the effort for you. |
@Mikey1848 I'm at work right now and will be until 4pm today. So if you wanted to stop by we could talk about things. Otherwise, we would have to talk Tuesday. |
@regexpressyourself I understand where you are coming from and thanks for being honest with me. |
Devpressed forum SIU counselling is only 6 dollars per session for the record. |
Description
As a user, I want to be able to export my flowcharts.
Acceptance Criteria
Priority
Must have
Time
3 days
The text was updated successfully, but these errors were encountered: