-
Notifications
You must be signed in to change notification settings - Fork 233
[cuegui] Add Request Core Buttons to MenuActions #1477
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
Merged
DiegoTavares
merged 6 commits into
AcademySoftwareFoundation:master
from
DiegoTavares:request_cores
Aug 21, 2024
Merged
[cuegui] Add Request Core Buttons to MenuActions #1477
DiegoTavares
merged 6 commits into
AcademySoftwareFoundation:master
from
DiegoTavares:request_cores
Aug 21, 2024
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Artists requested the ability to right click on a job and request cores, they wanted to formalize the process and have fields prepopulated for the artist, and the ability to add extra info.
|
Collaborator
Author
|
Roula's contribution to this PR happened while she was still at SPI. Even though she has signed the CLA, the check will fail as the email assigned to this commit is no longer associated with her GitHub account. |
Using qtpy instead
48721b2 to
72c84ef
Compare
When requesting cores, the CC field contains a default list of recepients that now can be configured using a pattern on cuegui.yaml 'email.show_support_cc_template'.
72c84ef to
e15f739
Compare
ramonfigueiredo
approved these changes
Aug 20, 2024
Collaborator
ramonfigueiredo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved with minor changes!
DiegoTavares
commented
Aug 21, 2024
PST has been replaced with Support Team Signed-off-by: Diego Tavares <[email protected]>
Signed-off-by: Diego Tavares <[email protected]>
Signed-off-by: Diego Tavares <[email protected]>
26b3b21 to
c861796
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This new feature adds a new menu action that opens a new
dialogto request more cores via email for a given job.The
dialogis an Email composed where the message is pre-populated with relevant data about the job. Fields, liketo,ccandbcc, need to be filled manually. The content of the message is editable.