Details
| Participant | Role | Time Spent | Comments | Latest Comment |
|---|---|---|---|---|
| Author | 9m | 2 | All but one was done already in the schematic tool, so I ... | |
| Moderator | ||||
| Reviewer - 0% complete | ||||
| Reviewer - 100% complete | 17m | 2 | I haven't checked if these changes have been made in the ... | |
| Reviewer - 0% complete | ||||
| Reviewer - 0% complete | 2m | 1 | How did you get these? These properties were pulled from ... | |
| Reviewer - 0% complete | ||||
| Reviewer - 0% complete | ||||
| Reviewer - 0% complete | 0m | |||
| Total | 28m | 5 |
- Files: 12
-
Patches:
- patch.txt: (anchored to swgemu : core3/branches/unstable/MMOCoreORB/ )
General Comments
01 Feb
Loshult says:
Can you do these changes in the crafting tool and verify that the changes you do have the desired effect?
03 Feb
Bert says:
I found that these changes were done in the schematic tool, so does that mean that they will change next update? They do have the desired effect when I tested them. If they were changed in the schematic tool and this isn't needed, then I can abandon the review.
03 Feb
Loshult says:
I haven't checked if these changes have been made in the schematic tool, but I don't think they have since they should have been committed then. Please do the change in the schematic tool as well so that these changes will stay even if we do a format change of the lua files later and exports every schematic again. Get hold of me on IRC if you need access to the schematic too.
04 Feb
Bert says:
All but one was done already in the schematic tool, so I then changed that one to match. All set.
How did you get these? These properties were pulled from the client iirc.
EDIT: Oh, I see... I'm afraid to edit these values like this since I'm not sure how this will interfere with kyle's crafting system.