Skip to content
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

HUGE update to Dungeon Fantasy #235

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

mirtexxan
Copy link
Contributor

@mirtexxan mirtexxan commented Jul 25, 2023

Changelog:

Basic:
Equipment: Added some missing equipments (all of the Clothing Items and the Scribing Kit)
Trait: Included some missing modifiers and options from the DF books.

DF1:
Added all the Equipment modifiers from DF1.
Refactored and cleaned all the DF1 templates, so that now they have the same layout.
Added all of the missing loadouts from DF13.
EDIT: Moved the Duelist and Sorcerer class to DFD:Swasbucklers and Pyramid 82, respectively.

DF4:
Added the missing Scholar template.
Refactored and cleaned all the templates, so that now they have the same layout of the DF1 ones.
Added loadouts to both Artificier and Scholar.
Added the Artificer lenses (the Scholar one are still missing).

DF9:
Refactored and cleaned all the DF9 templates, so that now they have the same layout of the DF1 ones.
Added all of the missing loadouts from DF13.
(Note: for some reasons the old commits you've already pulled, concering the missing templates from DF9 seem to still be visible in this pull request: don't know why).

DF10:
Added all of the missing loadouts from DF13 to the Innkeeper template.
Refactored and cleaned all the templates, so that now they have the same layout of the DF1 ones.

DF12:
Added all the equipment from DF12 to the library.
Added all of the missing loadouts from DF13.
Refactored and cleaned all the templates, so that now they have the same layout of the DF1 ones.

DF13:
Added all the equipment from DF13 to the library.

@mirtexxan mirtexxan changed the title HUGE updato to Dungeon Fantasy HUGE update to Dungeon Fantasy Jul 25, 2023
Copy link
Collaborator

@rinickolous rinickolous left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed Basic Set additions, see comments for details. Didn't go through meticulously, will come back to this later

Library/Basic Set/Basic Set Equipment.eqp Outdated Show resolved Hide resolved
Library/Basic Set/Basic Set Traits.adq Outdated Show resolved Hide resolved
Library/Basic Set/Basic Set Traits.adq Outdated Show resolved Hide resolved
Changed the description of the wardrobe equipment to avoid Copyright issues.
Added the correct prerequisites to Modular Abilities(Book-Learned Wisdom.)
Fixed the wrong page reference in Book-Learned Wisdom.
Also fixed the page reference code in Duelist.
},
"notes": {
"compare": "does_not_contain",
"qualifier": "None"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't check for a language where the character has native literacy as descibed in DF4. It checks for a language where the character is neither completely illiterate, nor completely incapable of speaking.

Instead of does not contain "None" it should be looking for does contain "Written (Native)"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right. Fixed.

@richardwilkes
Copy link
Owner

The branch is reporting conflicts in files you previously committed. I'm guessing you didn't start a new branch and instead continued on with the branch you used for the previous PR. That usually causes this sort of problem. Depending on the actual conflicts, it might be easier to squirrel these changes away, nuke your old repo, fork the master library again, apply your saved changes, then create a new PR.

@mirtexxan
Copy link
Contributor Author

mirtexxan commented Jul 29, 2023 via email

@richardwilkes
Copy link
Owner

I'm happy to merge this once the conflicts have been resolved.

@mirtexxan
Copy link
Contributor Author

mirtexxan commented Oct 11, 2023 via email

@richardwilkes
Copy link
Owner

I'm afraid the current conflicts are too severe for me to know what should be done. The suggestion I made on July 28 is probably the only way to resolve this correctly.

@mirtexxan
Copy link
Contributor Author

Fine, I'll be sure to do it in the following days :)

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

Successfully merging this pull request may close these issues.

4 participants