Skip to content

/Cataclysm-DDA

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

Clean up arrow recipes #14324

Merged
merged 7 commits into from Dec 8, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@Rail-Runner @Coolthulhu @chaosvolt @Rivet-the-Zombie
@Rail-Runner
Copy link
Contributor

commented Dec 6, 2015

  1. Change arrow making process to skip recipe that creates an unfletched arrow. Now these recipes create an arrow that is ready for being fired from a bow.
  2. Reduce amount of superglue and bone glue used in arrow recipes, to make it actually worth using it as an arrow component.
  3. Make most arrow recipes non-reversible.
  4. Remove "NO_RECOVER" flag from components in most recipes, as they aren't reversible anyway.

Rail-Runner added some commits Dec 6, 2015

@Rail-Runner
Merge remote-tracking branch 'refs/remotes/CleverRaven/master'
362ffbf
@Rail-Runner
Clean up arrow recipes. 
1. Change arrow recipes to skip recipe that creates an unfletched arrow.
Now they create an arrow that is ready for being fired from a bow.
2. Reduce amount of superglue and bone glue used in arrow recipes, to
make it actually worth using  it as an arrow component.
3. Make most arrow recipes non-reversible.
4. Remove "NO_RECOVER" flag from components in most recipes, as they
aren't reversible anyway.
cf701b6
@Rail-Runner
Fix (missed something)
Loading status checks…
68981c4
@Rail-Runner
Further reduce required amount of superglue.
Loading status checks…
c9eeb46
@Rail-Runner
Update More Survival Tools too.
Loading status checks…
4a5bada
@Rail-Runner
Combine all recipes to make arrow components into one recipe.
e588507
@Rail-Runner
Update More Survival Tools. 
Also tweak item group to prevent arrowheads from spawning.
Loading status checks…
045c602
@Coolthulhu

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Dec 6, 2015

Good changes.
Those glue costs could be lowered to 1 in all cases, though. Old costs were insane, but 20 thread is still much less than 1 glue.
@chaosvolt

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Dec 6, 2015

Ah, nice. I'd always favored the standard wooden arrows for being a tad simpler to make.

Though wooden crossbow bolts remain my favorite for now. XP
@Coolthulhu

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Dec 6, 2015

By the way, wouldn't it be possible to also "inline" fletching like that? Having only the final recipes for arrows would make them significantly less annoying to craft, maybe with the exception of fire-hardened ones.
@chaosvolt

This comment has been minimized.

Sign in to view
Copy link
Contributor

commented Dec 6, 2015

Hmm? Inline fletching? Might be feasible.
@Rail-Runner

This comment has been minimized.

Sign in to view
Copy link
Contributor Author

commented Dec 6, 2015

Done. You need to craft only once to get usable arrows now.

Rivet-the-Zombie added a commit that referenced this pull request Dec 8, 2015

@Rivet-the-Zombie
Merge pull request #14324 from Darkyhard/arrows 
Clean up arrow recipes
1db3202

@Rivet-the-Zombie Rivet-the-Zombie merged commit 1db3202 into CleverRaven:master Dec 8, 2015
1 check passed

1 check passed

default
Details

@Rail-Runner Rail-Runner deleted the Rail-Runner:arrows branch Dec 20, 2015

@cainiaowu cainiaowu referenced this pull request Nov 1, 2017

Merged

Arrow item cleanup #22325

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.