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

Initial release of Sky Fortress Invasion Instance (Official Conversion) #3586

Merged
merged 40 commits into from
Mar 8, 2021

Conversation

Atemo
Copy link
Contributor

@Atemo Atemo commented Oct 16, 2018

  • Addressed Issue(s): -
  • Server Mode: renewal
  • Description of Pull Request:
    TODO

Thanks to @admkakaroto and https://www.divine-pride.net/ for testing and some mobs infos

db/re/item_db.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
npc/re/instances/SkyFortrest.txt Outdated Show resolved Hide resolved
aleos89 and others added 3 commits October 17, 2018 15:56
Thanks to @aleos89 !
…ena/rathena into script/skyfortrest_instance

# Conflicts:
#	npc/re/instances/SkyFortrest.txt
@mjonrest
Copy link

#3084 i saw this issue, can this also release??

@AsurielRO
Copy link

just a missing item will be drop and monster skills all over are fine

@dev1-juan
Copy link

what seems to be the last problem for it to be released?

@Everade
Copy link
Contributor

Everade commented Dec 29, 2018

Still missing are the Random Options on drop:
https://irowiki.org/wiki/Vicious_Mind_Weapon_Enchants

And there are missing "" in the Stephen J. E. Card
Should be:
31026,Stephen_Jack_Ernest_Wolf_Card,Stephen Jack Ernest Wolf Card,6,20,,10,,,,,,,,64,,,,,{ bonus5 bAutoSpellWhenHit,"SO_FIREWALK",5,35,BF_WEAPON,0; bonus5 bAutoSpellWhenHit,"SO_ELECTRICWALK",5,80,BF_MAGIC,0; autobonus "{ bonus bFlee,200; }",30,10000,BF_WEAPON; autobonus "{ bonus bSpeedRate,25; }",25,10000,BF_MAGIC; /* unknown rate and specialeffect */ },{},{}

Lemongrass3110 and others added 9 commits January 29, 2019 16:49
# Conflicts:
#	db/re/item_combo_db.txt
#	db/re/mob_db.txt
* The quest log shouldn't change when using a ticket to enter
* Added a check to warp back to his map entrance when player exit the instance
Updated the script using some news script commands
* Corrects the Sky Fortress Key Keeper spawn location and rates
* Enabled 'Sky Fortress Treasure Box' / Wind Ghost in Experiment NPC when the Immortal Death Shadow are spawn
db/re/mob_db.txt Outdated
//3483,AS_EVIL_SHADOW3
//3484,AS_D_RAGGED_GOLEM
//3485,AS_D_CURSED_SOLDIER
3473,AS_RAGGED_GOLEM,Stefan.J.E.Wolf,Stefan.J.E.Wolf,160,20000000,1,1000000,1000000,1,1307,103,146,30,189,25,125,45,104,33,10,12,2,1,89,0x6283695,300,1008,1200,540,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,31026,1
Copy link
Contributor

Choose a reason for hiding this comment

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

seems like the monsters didnt have the proper item drop yet.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've created a PR to amend this based on information from Divine-Pride and RMS DBs

db/re/item_db.txt Outdated Show resolved Hide resolved
Atemo added a commit to Atemo/rathena that referenced this pull request Oct 24, 2020
Atemo added a commit that referenced this pull request Oct 25, 2020
* Implemented the items from sky fortress instance (part of #3586)

Co-authored-by: Lemongrass3110 <lemongrass@kstp.at>
src/map/skill.cpp Outdated Show resolved Hide resolved
db/re/skill_db.yml Outdated Show resolved Hide resolved
db/re/skill_db.yml Outdated Show resolved Hide resolved
src/map/battle.cpp Outdated Show resolved Hide resolved
@Atemo Atemo merged commit 95c671a into master Mar 8, 2021
@Atemo Atemo deleted the script/skyfortrest_instance branch March 8, 2021 14:12
@Atemo Atemo changed the title Initial release of Sky Fortress Invasion Instance (Walkthrough Conversion) Initial release of Sky Fortress Invasion Instance (Official Conversion) Mar 8, 2021
@Lemongrass3110 Lemongrass3110 mentioned this pull request Jan 16, 2023
81 tasks
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.

None yet