ZTL-ARTCC / Current_Website

vZTL's Official Website
GNU Affero General Public License v3.0
5 stars 6 forks source link

#672 Training Ticket All Drafts View #688

Closed iccowan closed 1 day ago

iccowan commented 2 weeks ago

This PR will:

2 Questions Before Merging:

kjporter commented 1 week ago

To address your two questions:

  1. Should we allow all training staff to see this view? --Hmm... let's populate the list as follows:
    • If the user is the TA, ATA, or senior staff, show all drafts here
    • If the user is training staff, but not one of the above, only this control and drafts that the present user created
  2. Should we add a button when viewing training tickets to go back to all tickets instead of going back to the user's tickets for the one that we're viewing? --Oh, I see what you mean here, that isn't the best behavior when you access a ticket from an area other than the user's training profile. Yes, I think you should add that button.
iccowan commented 5 days ago

To address your two questions:

  1. Should we allow all training staff to see this view? --Hmm... let's populate the list as follows:
  • If the user is the TA, ATA, or senior staff, show all drafts here
  • If the user is training staff, but not one of the above, only this control and drafts that the present user created
  1. Should we add a button when viewing training tickets to go back to all tickets instead of going back to the user's tickets for the one that we're viewing? --Oh, I see what you mean here, that isn't the best behavior when you access a ticket from an area other than the user's training profile. Yes, I think you should add that button.

Resolved both of these and squashed into 088cb0f