Skip to content

Courtier tech description not transferring to char sheet #364

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

Open
Tepedinoo opened this issue Feb 20, 2025 · 6 comments
Open

Courtier tech description not transferring to char sheet #364

Tepedinoo opened this issue Feb 20, 2025 · 6 comments

Comments

@Tepedinoo
Copy link

Heya! So, I noticed all classes' tech go to the character sheet... except for the courtier. I've been fiddling with the export, but to no success. Any chance I can get help with that? Thanks!

@OpenNingia
Copy link
Owner

The problem lies in the sheet_courier.pdf template. The fillable text field for the tech effect is wrong but I don't have the software to edit the pdf anymore. If you do have Adobe suite you could edit the pdf to make it similar to the bushi one then I could integrate it in the next release.

The bushi one is like this:

Image

while the courtier one is:

Image

For the bushi sheet the code fill the tech description in the field BUSHI_TECH_TEXT.x.y where x is the rank of the technique (starting from 0) and y is the index of the school (i.e. first school = 0, second school = 1, etc...)

The courier sheet should work the same and have a text field COURIER_TECH_TEXT.x.y or similar.

Let me know if you're able to help.

@Tepedinoo
Copy link
Author

Tepedinoo commented Feb 21, 2025

Is it possible to change the code just so the courtier technique goes into the bushi sheet? We got coords for that, right? I just can't find where to tell the code which sheet to use

@Tepedinoo
Copy link
Author

I also noticed now that, for some reason, ninja sheets do not come with techs

@OpenNingia
Copy link
Owner

Is it possible to change the code just so the courtier technique goes into the bushi sheet? We got coords for that, right? I just can't find where to tell the code which sheet to use

Yes, of course, however you would see "Bushi Schools" instead of "Courtier Schools" in the header.

The relevant code is here:

@Tepedinoo
Copy link
Author

Tepedinoo commented Feb 21, 2025

So, I fixed the ninja, all I had to do is to move the is_ninja to the elif above

However, no matter what I do, it keeps the courter sheet. I'm pretty noobish at programming, so I don't know what I'm doing wrong.

It feels like there is something else binding the sheet to courtiers

@OpenNingia
Copy link
Owner

So, I fixed the ninja, all I had to do is to move the is_ninja to the elif above

However, no matter what I do, it keeps the courter sheet. I'm pretty noobish at programming, so I don't know what I'm doing wrong.

It feels like there is something else binding the sheet to courtiers

Can you try with this branch? https://github.com/OpenNingia/l5r-character-manager-3/tree/fix_coutier_pdf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants