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

Added some missing events. #2303

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

Conversation

kisuka
Copy link
Contributor

@kisuka kisuka commented Oct 22, 2018

Pull Request Prelude

Changes Proposed

Affected Branches:

  • (Master)

Issues addressed:

Surprisingly I'm not completely done contributing to this project. :p

I took some time this last week or so to go through some of the missing event scripts as well as fix some stuff up.

This Pull Requests does the following:

  • Adds 2006 Xmas event script.
  • Adds 2007 Xmas event script.
  • Adds 2009 Xmas event script.
  • Adds 2010 Xmas event script.
  • Adds 2010 Valetine's Day event script.
  • Adds 2010 April Fool's Day event script.
  • Properly dates the existing Xmas, Whiteday, and Valetine's Day scripts.

The dates for the 2004 scripts were confirmed by looking through jAthena files and comparing the quest's contents with event posts from jRO, iRO, and kRO.

I'll be back with more scripts later.

Known Issues and TODO List

@kisuka kisuka force-pushed the missing-events branch 2 times, most recently from c4e1081 to f827737 Compare October 22, 2018 05:56
@HerculesWSAPI
Copy link
Contributor

This change is Reviewable

npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
npc/events/aprilfools_2010.txt Outdated Show resolved Hide resolved
@kisuka kisuka force-pushed the missing-events branch 4 times, most recently from 8d5cd28 to a874d6d Compare October 26, 2018 20:50
@Emistry Emistry added the component:scripts Affecting the scripts and NPCs label Dec 14, 2018
npc/events/christmas_2006.txt Outdated Show resolved Hide resolved
npc/events/christmas_2006.txt Outdated Show resolved Hide resolved
npc/events/christmas_2006.txt Outdated Show resolved Hide resolved
@kisuka kisuka requested a review from dastgirp May 22, 2019 19:24
Copy link
Member

@Emistry Emistry left a comment

Choose a reason for hiding this comment

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

the github suggestion tag seem like having bug, it keep point at the wrong suggestion .

npc/events/christmas_2007.txt Outdated Show resolved Hide resolved
npc/events/christmas_2007.txt Outdated Show resolved Hide resolved
npc/events/christmas_2006.txt Outdated Show resolved Hide resolved
npc/events/christmas_2006.txt Outdated Show resolved Hide resolved
npc/events/christmas_2007.txt Outdated Show resolved Hide resolved
npc/re/events/christmas_2011.txt Outdated Show resolved Hide resolved
npc/re/events/christmas_2011.txt Outdated Show resolved Hide resolved
npc/re/events/christmas_2011.txt Outdated Show resolved Hide resolved
npc/re/events/christmas_2011.txt Show resolved Hide resolved
npc/re/events/christmas_2011.txt Outdated Show resolved Hide resolved
Copy link
Member

@Emistry Emistry left a comment

Choose a reason for hiding this comment

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

  • review

@MishimaHaruna MishimaHaruna added this to the Release v2020.01.12 milestone Jan 12, 2020
Copy link
Member

@MishimaHaruna MishimaHaruna left a comment

Choose a reason for hiding this comment

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

I started to rebase, split into commits and fix the style, but I give up -- please fix your conversion script...

mes("... You didn't tell me what I to bring...");
next();
mes("[Ohholy]");
mes("No way! I said ^0000ff 1 Broken Jewel , 3 Holy Water, 1 Red Gemstone, and 1 Yellow Gemstone^000000 clearly.")
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't really do what it may seem it's doing

@MishimaHaruna MishimaHaruna added the codereview:needsedits Some edits have been requested before the pull request can be accepted label Jan 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codereview:needsedits Some edits have been requested before the pull request can be accepted component:scripts Affecting the scripts and NPCs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants