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.

Sign up for GitHub

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

Jump to bottom

You can see the reward of seers hut if AI visited it. #1952

Closed
krs0 opened this issue Apr 12, 2023 · 5 comments · Fixed by #3035
Closed

You can see the reward of seers hut if AI visited it. #1952

krs0 opened this issue Apr 12, 2023 · 5 comments · Fixed by #3035
Labels
bug visibility Data visibile to players

Comments

@krs0
Copy link
Collaborator

krs0 commented Apr 12, 2023

Describe the bug
When you hover on map a seers hut visited by AI, you will see the reward.

To Reproduce
Steps to reproduce the behavior:

  1. Start All for One as Blue!!!
  2. vcmieagles
  3. End 1-2 turns
  4. Hoover on Reds seers hut left or red mains castle

Screenshots
image

Reported here: https://bugs.vcmi.eu/view.php?id=2856
@krs0 krs0 added the bug label Apr 12, 2023
@dydzio0614
Copy link
Member

also "defeat ring" text doesn't make sense

@IvanSavenko IvanSavenko added this to the Release 1.2 milestone Apr 13, 2023
@IvanSavenko
Copy link
Member

  • Will fix tooltip for 1.2 (core.seerhut.names and defeating rings)
  • AI visiting fix goes to 1.3

@IvanSavenko IvanSavenko modified the milestones: Release 1.2, Release 1.3 Apr 13, 2023
@krs0
Copy link
Collaborator Author

krs0 commented Apr 13, 2023

Splitted the UI part.
#1972

@Alucard648
Copy link

Is the revealing of seer hut quest intentional, as allied hero visits it?

@krs0
Copy link
Collaborator Author

krs0 commented Apr 13, 2023

In this particular case that's the enemies seers hut quest. But good point, allies should see your quests.

@IvanSavenko IvanSavenko mentioned this issue Jul 13, 2023
Merged
11 tasks
@IvanSavenko IvanSavenko removed this from the Release 1.3 milestone Jul 22, 2023
@DjWarmonger DjWarmonger added the visibility Data visibile to players label Jul 23, 2023
@IvanSavenko IvanSavenko mentioned this issue Oct 12, 2023
Merged
@Nordsoft91 Nordsoft91 linked a pull request Oct 12, 2023 that will close this issue
Quests rework #3035
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug visibility Data visibile to players
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Quests rework Nordsoft91/vcmi
5 participants
@IvanSavenko @DjWarmonger @Alucard648 @dydzio0614 @krs0