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

@warriorjs/tower-tick-tick-boom warrior haven't got the "pivot" ability and bashing 'backward' forcedly #242

Open
dandgerson opened this issue Oct 13, 2018 · 0 comments
Labels
area: towers Related to towers status: has pr Issues with an accompanying Pull Request type: enhancement A potential improvement to something that already exists

Comments

@dandgerson
Copy link

dandgerson commented Oct 13, 2018

Hey! WarriorJS,

When my JS Warrior attacking in 'backward' direction he takes only 3 points damage, while if he will be able pivot to the enemy face to face he will cuts 5 points for each attack. And this is not cheat, because it's normal, when you want to bash or slash the enemy's skull, you have to pivot yourself to face him))

and i was really puzzled when my Warrior was surrounded by the enemies on third level of the tower and my EAST - faced Warrior bashing 'backward' the Sludge, and he couldn't turn to face him with his "pivot" ability.
warriorjs/packages/warriorjs-tower-tick-tick-boom/src/index.js

import {
  attack,
  bind,
  detonate,
  directionOf,
  directionOfStairs,
  distanceOf,
  feel,
  health,
  listen,
  look,
  maxHealth,
  pivot, //  add "pivot" ability 
  rescue,
  rest,
  think,
  walk,
} from '@warriorjs/abilities';
warrior: { // warrior of the third level
          ...Warrior,
          position: {
            x: 1,
            y: 1,
            facing: EAST,
          },
          abilities: {
            bind: bind(),
            pivot: pivot(), // i think it should be like this), but i assume that "pivot" must have on first level
            rescue: rescue(),
          },
        },
@dandgerson dandgerson changed the title @warriorjs/tower-tick-tick-boom why not includes the warrior's "pivot" ability ? @warriorjs/tower-tick-tick-boom warrior haven't got the "pivot" ability and bashing 'backward' forcedly Oct 13, 2018
dandgerson added a commit to dandgerson/warriorjs that referenced this issue Oct 13, 2018
By issue olistic#242 @warriorjs/tower-tick-tick-boom warrior haven't got the "pivot" ability and bashing 'backward' forcedly
olistic#242
@olistic olistic added area: towers Related to towers type: enhancement A potential improvement to something that already exists status: has pr Issues with an accompanying Pull Request labels Oct 16, 2018
olistic pushed a commit to dandgerson/warriorjs that referenced this issue Apr 4, 2023
By issue olistic#242 @warriorjs/tower-tick-tick-boom warrior haven't got the "pivot" ability and bashing 'backward' forcedly
olistic#242
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: towers Related to towers status: has pr Issues with an accompanying Pull Request type: enhancement A potential improvement to something that already exists
Projects
None yet
Development

No branches or pull requests

2 participants