-
Notifications
You must be signed in to change notification settings - Fork 71
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
Add affiliations update button #582
base: master
Are you sure you want to change the base?
Conversation
What about putting next to the user button? So we're keeping controls all together. |
Move affiliation button to right of user button (Button is slightly larger than user button, UI aesthetically better to have on right.)
Further work is needed on this. Do not push this to master. |
As previsouly explained, button is containg "setup" - so when you hit it, it will know what to do and which date to send based on a context. This context is provided by the data attributes.
When you click on it, the following ajax payload is sent against SeAT {
"type": "character",
"entity_id": 0,
"job_name" : "Job\\FQCN"
} Lines 103 to 126 in ad943d2
Which is processed by this backend web/src/Http/Controllers/Tools/JobController.php Lines 55 to 84 in 5907741
You will need to put a Also, it might need a bit refactor to be able to use affiliation job since as you noticed, all currently used jobs only took a character ID together with a token as parameter - however, the affiliation job is a bucket and will need an array of stuff. |
Hey @veteranmina , what's the state of this PR |
Change would add button in summary card to look like this. If you believe a better location would be rather elsewhere let me know 😄