Navigation

    TripleA Logo

    TripleA Forum

    • Register
    • Login
    • Search
    • TripleA Website
    • Categories
    • Recent
    • Popular
    • Users
    • Groups
    • Tags

    Feature Request: Reinstate Internal Dice Roller

    Feature Requests & Ideas
    3
    5
    709
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • Panther
      Panther Admin Moderators last edited by

      Github sends me here, so I request to reinstate the Internal Dice Roller for Play By Forum games and Play by Email games, as discussed here:

      https://forums.triplea-game.org/topic/187/internal-dice-sender-cleanup

      That feature has been removed at any time during development from 1.9.0.0.13066 to the current version. I cannot tell exactly when as I did not find a pull request explicitly addressing this.

      1 Reply Last reply Reply Quote 2
      • RoiEX
        RoiEX Admin last edited by

        Found the PR https://github.com/triplea-game/triplea/pull/4454
        Turns out I removed the class, I can't remember why though 🤔
        I believe there was a discussion of who was still using it but I don't remember if it was on GitHub or on this forum.

        Adding it back it should be a managable task, but not as simple as resurrecting the class because a lot of code changed in the meantime.

        RoiEXLab created this issue in triplea-game/triplea

        closed Refactor PbF/PbEM Poster code #4454

        Panther 1 Reply Last reply Reply Quote 1
        • Panther
          Panther Admin Moderators @RoiEX last edited by Panther

          @RoiEX Thank you for digging this out. Maybe the "discussion" was in the topic I quoted above. However I must have missed any related discussion unfortunately. And as rarely anyone on the forums used any newer version than 1.9.x this change did not pop up earlier.

          I have no idea about the coding implications of course. But in the end it would be sufficient if the "local game" dice logic was used in case the dice server settings are not filled in.
          At the moment I cannot start the game when leaving the dice server settings empty.

          Just an idea...

          1 Reply Last reply Reply Quote 2
          • RoiEX
            RoiEX Admin last edited by

            But in the end it would be sufficient if the "local game" dice logic was used in case the dice server settings are not filled in.

            That would be a good option to consider

            1 Reply Last reply Reply Quote 1
            • LaFayette
              LaFayette Admin last edited by

              Perhaps this is simple as making the dice roller fields optional. (though, if you set one, then you need to set all of them as required, or leave them all blank to not use a dice roller). To make the UX better for this we could default a checkbox "Use MARTI dice roller" to checked, and if you uncheck it then it hides the MARTI dice roller fields. A Help button would be pretty key here to explain that MARTI would do external dice-rolls and internal would simply just not use that.

              I think there are probably some implications for the game engine itself as the way PBF is done is kinda hacky, the flag indicating PBF is not direct and probably would trigger a request to Marti. We'd need to make the 'using-marti' be an (internal) game option and to check only that for whether to use marti or not.

              1 Reply Last reply Reply Quote 1
              • 1 / 1
              • First post
                Last post
              Copyright © 2016-2018 TripleA-Devs | Powered by NodeBB Forums