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

Fixed bug #21486: Team label clears a non-team label #94

Merged
merged 3 commits into from
Feb 25, 2014

Conversation

sachith500
Copy link
Contributor

Editing a non-team label to be a team label no longer removes the old label (ie, everyone not in that team can still see it).

https://gna.org/bugs/index.php?21486

Signed-off-by: Sachith Hasaranga Seneviratne sachith500@gmail.com

Editing a non-team label to be a team label no longer removes the old label (ie, everyone not in that team can still see it).

https://gna.org/bugs/index.php?21486

Signed-off-by: hp <sachith500@ubuntu.(none)>
@cbeck88
Copy link
Member

cbeck88 commented Feb 17, 2014

Just a quick question about this -- what is the issue here? You don't want that Team A can get information even about the locations of Team B's private labels?

@sachith500
Copy link
Contributor Author

Team labels can be used to clear non-team labels on the map. For example, if we're playing a game that requires borders be set up (due to NAPs etc, in modes like conquest), you might have non-team labels saying "Border", but for your team (or yourself alone), you might have the need to place a team label on that spot without removing the non-team (global) label. I picked this up from the bug tracker (I wasn't the one who submitted the ticket).

@Ja-MiT
Copy link
Contributor

Ja-MiT commented Feb 23, 2014

Are you listed in data/core/about.cfg?
Could you add entries in changelog and players_changelog?

Otherwise, this looks good to go.

@sachith500
Copy link
Contributor Author

Yeah, I'll add them and update this PR

Added changelog entries and updated about.cfg
changelog.

Signed-off-by: Sachith Hasaranga Seneviratne <sachith500@gmail.com>
@sachith500
Copy link
Contributor Author

OK, done. Btw, I had to send an additional commit to fix a typo in the previous one. If you want me to rebase and send a new PR, let me know.

@sachith500
Copy link
Contributor Author

yeah :P
I've gotten used to adding -s to my commits. mistakenly wrote the commit message below it :S

@AI0867 AI0867 merged commit ebd111b into wesnoth:master Feb 25, 2014
irydacea added a commit that referenced this pull request Mar 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants