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

create Soul reaper job change quest #8159

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

Conversation

BotGigo
Copy link
Contributor

@BotGigo BotGigo commented Mar 7, 2024

  • Addressed Issue(s):

Missing quest for soul reaper job.

  • Server Mode:

Renewal.

  • Description of Pull Request:

Create the files in db for quest and instace for the soul repaer job change quest and add the script for the quest.

Correct comparations in conditions for some base level achievements to work with the multi level check of achievements.
Add the quest and the instance to the db for the soul reaper job quest.
Add soul reaper job change quest.
@Atemo
Copy link
Contributor

Atemo commented Mar 9, 2024

Thanks you for your work. Before any review I have a few questions.
Was your work done based on replay?
Can you add a comment later where you added your custom dialog?

@BotGigo
Copy link
Contributor Author

BotGigo commented Mar 9, 2024

Thanks you for your work. Before any review I have a few questions. Was your work done based on replay? Can you add a comment later where you added your custom dialog?

Yes, the script is based on the video, is written in the script header, so the dialogs are the ones that appear in the video, and yes, lately I'll comment the fill dialogs .

@Atemo
Copy link
Contributor

Atemo commented Mar 9, 2024

Did you use a replay parser?

@BotGigo
Copy link
Contributor Author

BotGigo commented Mar 9, 2024

Did you use a replay parser?

What do you mean by that?

Edit: I just write everything by hand.

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

2 participants