Feat: Some Targuna Reviews and the Targuna Quest#784
Open
htc16 wants to merge 11 commits into
Open
Conversation
Collaborator
|
Where monsters file? |
Contributor
Author
Hold on to that PR, I'll change it and release everything together, including the Targuna quest! |
555b7d2 to
78fef16
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Targuna island section was easy; I added the monsters while playing Tibia. For the other areas, I used videos as a reference, so there might be some errors.
However, I believe everything is quite accurate, with perhaps only minor variations.
I should mention upfront that I couldn't do two things due to my technical limitations—since I'm not a programmer—and I'll list them below in case a kind soul can complete them.
1) In the
movements_main_continent_hint.luafile, I couldn't find or add the screen shown below that appears upon arriving at the island for the first time:This one is resolved THANKS TO @jprzimba
I did, however, mark the exact location where it should be placed, so we're counting on someone's help with this.
2) I noticed a new banner that wasn't there before—perhaps because I haven't played Tibia in a long time—but I found it very interesting. I even tried adding it, but it seemed too complex (I believe this banner was introduced in the Targuna update, though I'm not certain; I'm only seeing it now). It would be great if someone could add it; here is the banner I'm referring to:
This one is resolved in #795 THANKS TO @jprzimba and @Mckay666
This banner is dynamic and updates as spells are unlocked.
One more thing: if you test the boss and find that the spells—whether the monsters' or the boss's own—aren't quite right, please help me fix them; I don't think I handled those aspects as well as I could have. But if they meet the standards, then that's fine!
References (more YouTube videos not listed below):
https://tibia.fandom.com/wiki/Targuna_Quest
https://www.tibiawiki.com.br/wiki/Targuna_Quest