-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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 expanded 4ths to jobmaster #8267
base: master
Are you sure you want to change the base?
Conversation
@@ -504,6 +526,7 @@ OnInit: | |||
setarray .Req_Fourth[0],200,70; // Minimum base level, job level to change to fourth class | |||
setarray .Req_Exp_NJ_GS[0],99,70; // Minimum base level, job level to turn into Expanded Ninja and Gunslinger | |||
setarray .Req_Exp_SNOVI[0],99,99; // Minimum base level, job level to turn into Expanded Super Novice | |||
setarray .Req_SHandler[0],200,60; // Minimum base level, job level to turn into Spirit Handler |
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.
Not sure if these are the correct levels
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.
https://www.youtube.com/watch?v=kuCVZv21Amc
The person in this video is level 200/60.
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.
} | ||
|
||
if (.FourthExpanded && .@eac == EAJ_SUPER_NOVICE_E) { | ||
Require_Level(.Req_Fourth[0], .Req_Fourth[1]); |
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.
right levels?
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.
The required level (200/70) is correct
https://www.divine-pride.net/tools/skilltree/4307
Man do you will merge the missing jobs in main branch too? This: #7024 |
Oh, they're not all in yet? Then probably not, I'll look at the ones that are actually implemented |
The skills are missing, but the classes are implemented. |
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.
Good for me!
I had to guess some of the values