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

Bug Fix to allow tokens without sight to move #41

Closed
JamzTheMan opened this issue Dec 12, 2017 · 2 comments
Closed

Bug Fix to allow tokens without sight to move #41

JamzTheMan opened this issue Dec 12, 2017 · 2 comments
Assignees
Labels
bug fixed bug fix is committed for next release MapTool issue relates to MapTool code vs 3rd party libs

Comments

@JamzTheMan
Copy link
Owner

JamzTheMan commented Dec 12, 2017

Allow player owned tokens without "sight" to move within any area other player owned tokens can "see".

Reason/example: Wizard summoned spell token, ie Flaming Sphere, doesn't have sight but Wizard needs to be able to move token when Individual FoW is on.

@JamzTheMan JamzTheMan self-assigned this Dec 12, 2017
@JamzTheMan JamzTheMan added bug fixed bug fix is committed for next release MapTool issue relates to MapTool code vs 3rd party libs labels Dec 12, 2017
@JamzTheMan JamzTheMan changed the title Bug Fix Bug Fix to allow tokens without sight to move Dec 12, 2017
JamzTheMan added a commit that referenced this issue Dec 12, 2017
Task-Url: #41

Signed-off-by: Jamz <Jamz@Nerps.net>
@JamzTheMan
Copy link
Owner Author

Does not seem to be 100% fixed...

@JamzTheMan JamzTheMan reopened this Jul 2, 2018
@JamzTheMan JamzTheMan added investigating researching if issue is valid and/or reasonable and removed fixed bug fix is committed for next release labels Jul 2, 2018
JamzTheMan added a commit that referenced this issue Sep 2, 2018
 * Tokens without sight can move within current PC tokens field of view
when iFoW server options are checked
 * Tweaked sentry.properties to use dummy DSN when in development mode

Task-Url: #41

Signed-off-by: Jamz <Jamz@Nerps.net>
JamzTheMan added a commit that referenced this issue Sep 2, 2018
* #77: Drag path does not seem to be working correctly.

	* Tweaked the A* algorithm for more natural/straighter moves though
open spaces for both square and hex grids
	* Tweaked the A* algorithm to find shortest path more consistently

Task-Url: #77

Signed-off-by: Jamz <Jamz@Nerps.net>

* #77: Drag path does not seem to be working correctly.

 * Change log amended
 * Spotless applied

Task-Url: #77

Signed-off-by: Jamz <Jamz@Nerps.net>

* Minor update to PDF Extract

 * Updated PDFBox lib to latest
 * Sync'd PDF extract code from TokenTool (similiar but not 100% same on
purpose)

Signed-off-by: Jamz <Jamz@Nerps.net>

* #80 Comparison method violates its general contract in...

 * Reverting VisibleAreaSegment to previous version...

Signed-off-by: Jamz <Jamz@Nerps.net>

* #81: Cell Highlight distance text not sizing for grid sizes

 * Cell Highlight distance text now properly scales with grid size
 * Adjusted logging slightly

Task-Url: #81

Signed-off-by: Jamz <Jamz@Nerps.net>

* CI Updates

* CI Updates

* CI Updates

* CI Updates

* Gradle Process Updated

 * Vendor property now comes from gradle.properties
 * Version now comes from Git, uses SNAPSHOT-{commit} for
development/non-tagged releases and uses Tag name if commit is tagged
 * Updated Apache logging versions
 * sentry.properties file now generated, DSN is now only configured for
tagged releases, eg Production. This prevents sentry logging during
development

Signed-off-by: Jamz <Jamz@Nerps.net>

* #41: Bug Fix to allow tokens without sight to move

 * Tokens without sight can move within current PC tokens field of view
when iFoW server options are checked
 * Tweaked sentry.properties to use dummy DSN when in development mode

Task-Url: #41

Signed-off-by: Jamz <Jamz@Nerps.net>
@JamzTheMan JamzTheMan added fixed bug fix is committed for next release and removed investigating researching if issue is valid and/or reasonable labels Sep 6, 2018
@JamzTheMan
Copy link
Owner Author

Hopefully have this one nailed down now...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fixed bug fix is committed for next release MapTool issue relates to MapTool code vs 3rd party libs
Projects
None yet
Development

No branches or pull requests

1 participant