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

TFS update #302

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

TFS update #302

wants to merge 5 commits into from

Conversation

WibbenZ
Copy link
Contributor

@WibbenZ WibbenZ commented Mar 4, 2017

Closes #280
Started and no errors, might have missed something but will atleast decrease the amount of support threads :>

Started and no errors, might have missed something but will atleast
decrease the amount of support threads :>
@@ -462,7 +462,7 @@
<monster name="Charged Energy Elemental" file="electro-elementals/charged energy elemental.xml" />
<monster name="Energy Elemental" file="electro-elementals/energy elemental.xml" />
<monster name="Massive Energy Elemental" file="electro-elementals/massive energy elemental.xml" />
<monster name="Overcharged Energy Element" file="electro-elementals/overcharged energy element.xml" />
<monster name="Overcharged Energy Elemental" file="electro-elementals/overcharged energy elemental.xml" />
Copy link
Contributor

Choose a reason for hiding this comment

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

"Element" is correct

@WibbenZ
Copy link
Contributor Author

WibbenZ commented Mar 4, 2017

@TheSumm Changed it back, noticed that the problem is that an old map is downloaded (before you fixed this)
https://github.com/orts/world/tree/78c40f9b422e907916bb3ab245294dd0f58b9c3f

@pedroxs
Copy link

pedroxs commented Dec 19, 2017

@WibbenZ I just tried your PR with latest TFS (from master) and I got a few errors, but the most annoying is that no one is able to buy spells from NPCs. Do you have any ideia of what could be the reason? If you need any other info please ask. Thanks.

@WibbenZ
Copy link
Contributor Author

WibbenZ commented Dec 20, 2017

Because it's not updated ill update it when I got time.
Feel free to add your fixes.

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