[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Follow-up Comment #5, bug #14210 (project freeciv): Another rebase of the patch, this time to go over patches #4889 and #4990, and use the semantics enabled there. As a side effect, no longer advertises available stuff just because the discovered technologies don't block it. Still incomplete. (file #21562) ___ Additional Item Attachment: File name: notify-user-of-new-build-targets+rebase2.patch Size:9 KB ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Follow-up Comment #2, bug #14210 (project freeciv): An updated patch, using the new research_info packet. Still doesn't provide cache optimisation, or include the necessary client support to actually work, so still hoping for this ticket to be adopted. (file #21100) ___ Additional Item Attachment: File name: notify-user-of-new-build-targets+rebase.patch Size:9 KB ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Follow-up Comment #3, bug #14210 (project freeciv): Is there a reason to include this feature in server side too? I thought it could be implemented in client side only, to avoid network spams. ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Follow-up Comment #4, bug #14210 (project freeciv): The only reason my patch is implemented in the server is to parallel the E_NEW_GOVERNMENT code. Doing this (and the government notice) entirely on the client is probably better. ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Follow-up Comment #1, bug #14210 (project freeciv): The attached patch has received only the lightest testing, but should provide most of the required engine support for this feature. Performance could be improved with caching (see inline FIXME), and perhaps the nature of the messages should be split by type, rather than all using E_NEW_BUILD_TARGET. I have no idea how to do the client side of this, either the preferences, or adding proper support for the presented message, so hope someone else will pick up this ticket. (file #20635) ___ Additional Item Attachment: File name: notify-user-of-new-build-targets.patch Size:9 KB ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Update of bug #14210 (project freeciv): Planned Release: 2.4.0 = ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Update of bug #14210 (project freeciv): Planned Release: 2.3.0 = 2.4.0 ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
Update of bug #14210 (project freeciv): Planned Release: = 2.3.0 ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #14210] optionally include advance benefits in output message
URL: http://gna.org/bugs/?14210 Summary: optionally include advance benefits in output message Project: Freeciv Submitted by: nmx Submitted on: Friday 08/28/2009 at 15:24 Category: None Severity: 1 - Wish Priority: 5 - Normal Status: None Assigned to: None Originator Email: Open/Closed: Open Release: Discussion Lock: Any Operating System: None Planned Release: None ___ Details: When you gain an advance that enables a new form of government, the output message helpfully points out that a new form of government is available. You may not want to switch to it, but it's nice to know. I would love to see similar behavior for wonders, primarily, and improvements and units secondarily - especially if they were all optional, per the user's preference. For example, the following message would remain the same: Learned Bridge Building. Our scientists focus on Mysticism; goal is Theology. ...but this next message would get additional information: Learned Mysticism. Our scientists focus on Philosophy; goal is Theology. Mysticism enables you to build the Oracle wonder. ...something like that. More examples: Learned Masonry. Masonry enables you to build the Great Wall wonder and the Pyramids wonder. Masonry enables you to improve your cities with City Walls and Palace. Learned Seafaring. Seafaring enables you to improve your cities with Harbor. Seafaring enables you to build Explorer units. These options would be on by default, but could be individually toggled. ___ Reply to this item at: http://gna.org/bugs/?14210 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
