search

zak-hassan / WhoTeachsWhat

JBoss JSP Application for Data Management
0 stars 0 forks source link

Swift Report not using real data #169

Open ghost opened 10 years ago

ghost commented 10 years ago

We can't properly test the swift report if we're not using real data from our data base. So long as you are returning the same variable names, the UI will not be affected.

zak-hassan commented 10 years ago

I'll put in real data and test it out.

ghost commented 10 years ago

Also, could you make the weekly totals? I pushed in code with the variables set up. I have to make some documents and such.

zak-hassan commented 10 years ago

I will add another property to the swift object with the total so you have everything calculated.

On Wed, Apr 9, 2014 at 1:32 PM, Anil Santokhi notifications@github.comwrote:

Also, could you make the weekly totals? I pushed in code with the variables set up. I have to make some documents and such.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-39992657 .

zak-hassan commented 10 years ago

There will be some addition done in the UI that will require you to somehow add up all the totals and display it like in the screen shot.

On Wed, Apr 9, 2014 at 1:35 PM, Zakeria Hassan zak.hassan1010@gmail.comwrote:

I will add another property to the swift object with the total so you have everything calculated.

On Wed, Apr 9, 2014 at 1:32 PM, Anil Santokhi notifications@github.comwrote:

Also, could you make the weekly totals? I pushed in code with the variables set up. I have to make some documents and such.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-39992657 .

ghost commented 10 years ago

You don't need that. To a git pull and check the AJAX.

I already have the totals calculated. I just need you to print them out. I'm wrapped up in a lot of documents right now.

zak-hassan commented 10 years ago

Okay the code is pushed in that works can you run the application and tell me if its working for you.

On Wed, Apr 9, 2014 at 1:37 PM, Anil Santokhi notifications@github.comwrote:

You don't need that. To a git pull and check the AJAX.

I already have the totals calculated.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-39993202 .

ghost commented 10 years ago

I will have to reenter data since your last pushed wiped it. May I ask why we already have to reenter all of our data every time you make a change? Are you making changes to our models? It could affect the AJAX in the UI if you are.

zak-hassan commented 10 years ago

Hey Anil,

Just use the insert script to populate the data.

ghost commented 10 years ago

Has the insert script been updated?

On Wed, Apr 9, 2014 at 1:45 PM, Zak Hassan notifications@github.com wrote:

Hey Anil,

Just use the insert script to populate the data.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-39994195 .

ghost commented 10 years ago

It doesn't insert enough data. Some tables get no data. Are you generating it from the MySQL model file?

On Wed, Apr 9, 2014 at 1:45 PM, Anil Santokhi anil.d.santokhi@gmail.comwrote:

Has the insert script been updated?

On Wed, Apr 9, 2014 at 1:45 PM, Zak Hassan notifications@github.comwrote:

Hey Anil,

Just use the insert script to populate the data.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-39994195 .

  • Anil D. Santokhi

ghost commented 10 years ago

Faculty, FacultyToCourseInSemesterYear, ManageUsers ManageAccessLevel CourseInSemester and many others. I know I added all this into the MySQL model file. Please update this script. We can't have this going into the presentation.

zak-hassan commented 10 years ago

You will have to update the model file then you can generate a new script using mysql workbench. Then click File > Export > Select the first option . Make sure to check off generate insert statements.

ghost commented 10 years ago

Still uses dummy data.

zak-hassan commented 10 years ago

We can demo the dummy data for now and tell them that due to time limit we couldn't finish the rest. We can say that we setup the building blocks and all that is left is to pull the data but we couldn't test this feature fully. Unfortunately we ran out of time.

On Wed, Apr 9, 2014 at 3:04 PM, Anil Santokhi notifications@github.comwrote:

Still uses dummy data.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-40003532 .

ghost commented 10 years ago

Before we do that, can you please just enter real data into the application through the UI and test? We shouldn't let an INSERT statement decide if a feature can make it or not.

ghost commented 10 years ago

Remember, if I tell Emile it's not done, it's not done. That's it. So please make sure it cannot be done first.

zak-hassan commented 10 years ago

If we say can't be done. Then we do it. Its a good surprise. Better safe then sorry.

On Wed, Apr 9, 2014 at 3:46 PM, Anil Santokhi notifications@github.comwrote:

Remember, if I tell Emile it's not done, it's not done. That's it. So please make sure it cannot be done first.

Reply to this email directly or view it on GitHubhttps://github.com/zmhassan/WhoTeachsWhat/issues/169#issuecomment-40008140 .