YBNetwork / Issues-Enhancements

0 stars 0 forks source link

Coach's Dashboard - My Teams - View Team - "My Tournament(s) Status" #8

Closed kirkwhiteman1 closed 7 years ago

kirkwhiteman1 commented 7 years ago

We need to reformat the "My Tournament(s)" page. Please see attached document.

YBN Coach Dashboard MY TEAMS View Teams My Tournament(s) Status.xlsx

kirkwhiteman1 commented 7 years ago

Please note - I have added a couple data fields to this issues. The purp YBN Coach Dashboard MY TEAMS View Teams My Tournament(s) Status.xlsx ose of these changes is to allow the coach to print off the popup information. This print off will contain basically an invoice of sorts

bguerrettaz commented 7 years ago

Did I read this line correctly?

-When user (coach) clicks on View Tournament Details I would like this to Say "View Tournament Invoice"

So change the 'View Tournament Details' button to 'View Tournament Invoice'?

Just want to confirm.

kirkwhiteman1 commented 7 years ago

Correct

Kirk Whiteman

On May 2, 2017 7:26 PM, "bguerrettaz" notifications@github.com wrote:

Did I read this line correctly?

-When user (coach) clicks on View Tournament Details I would like this to Say "View Tournament Invoice"

So change the 'View Tournament Details' button to 'View Tournament Invoice'?

Just want to confirm.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/YBNetwork/Issues-Enhancements/issues/8#issuecomment-298797366, or mute the thread https://github.com/notifications/unsubscribe-auth/Aaxnf-4WX-3Pwvm1slEKAWwarnGV5fqCks5r18mdgaJpZM4NKryE .

bguerrettaz commented 7 years ago

Hi Kirk,

I'm making good progress on this issue. If you have a few moments, I'm hoping you can go out and validate what is currently done. Please go to the tournament status page and make sure that looks as desired. Key changes on that page:

image

The items highlighted in yellow is still work in progress, I need to bring in the necessary data and hook it to the proper fields. Does this look like what you're wanting overall?

Could of questions on that popup:

Please let me know your thoughts.

Thanks, Blake

kirkwhiteman1 commented 7 years ago

Yes. Looks great.

We are only showing 1 team.

Registration Due should actually be the data field from the director dahsboard (Registration Deadline)

Kirk Whiteman

On May 3, 2017 8:56 AM, "bguerrettaz" notifications@github.com wrote:

Hi Kirk,

I'm making good progress on this issue. If you have a few moments, I'm hoping you can go out and validate what is currently done. Please go to the tournament status page and make sure that looks as desired. Key changes on that page:

  • Cleaned up of some code issues
  • Renamed some headers
  • Renamed to 'View Tournament Invoice'
  • Made the special request text area 'hot', meaning, when you type something in, and hit the submit button, its saved to the database without doing a full page reload (using AJAX and jQuery)
  • Started work on the director/invoice popup:

[image: image] https://cloud.githubusercontent.com/assets/5695475/25663776/2f4d0e06-2fde-11e7-9d35-30aaf464b32f.png

The items highlighted in yellow is still work in progress, I need to bring in the necessary data and hook it to the proper fields. Does this look like what you're wanting overall?

Could of questions on that popup:

  • Can there be only 1 team listed, or is that 1 to many?
  • For the registration due date, how do you want that to be shown? If they are registered show 'Registered On' and if not, the date registration needs to be completed on? That was just confusing from the spec doc.

Please let me know your thoughts.

Thanks, Blake

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/YBNetwork/Issues-Enhancements/issues/8#issuecomment-298918315, or mute the thread https://github.com/notifications/unsubscribe-auth/Aaxnf17d3ir5MaFsmBp7zaCZwRZCAPqeks5r2IeZgaJpZM4NKryE .

bguerrettaz commented 7 years ago

Hi Kirk - ok, I think I have everything in line with your requirements. Can you please take a look over everything that this issues encompasses and let me know your thoughts? If its complete with no changes, then we can close this issue (it gets archived and can always be reopened).

Thanks, Blake

kirkwhiteman1 commented 7 years ago

Just checked it out and looks good to close.

kirkwhiteman1 commented 7 years ago

Confirmed

Kirk Whiteman

On May 3, 2017 7:05 PM, "bguerrettaz" notifications@github.com wrote:

Closed #8 https://github.com/YBNetwork/Issues-Enhancements/issues/8.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/YBNetwork/Issues-Enhancements/issues/8#event-1068183723, or mute the thread https://github.com/notifications/unsubscribe-auth/Aaxnf4pI9V8iAjspYxcatdhICnQJfGh2ks5r2RZigaJpZM4NKryE .