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

Target History (Max hist-target cleanup, wrong order) #2

Closed
kawaiipantsu opened this issue Mar 12, 2022 · 0 comments · Fixed by #3
Closed

Target History (Max hist-target cleanup, wrong order) #2

kawaiipantsu opened this issue Mar 12, 2022 · 0 comments · Fixed by #3
Assignees
@kawaiipantsu
Labels
bug Something isn't working

Comments

@kawaiipantsu
Copy link
Owner

When adding more than 50 (default) targets next time you start Redjoust it's supposed to clean up the history list of old targets.
Removing so there is only the last 50 targets saved. Obviously it should remove old-targets first :)

Behavior expected / Reproduce

  • Add "target 1"
  • Add 49 more
  • Add "target 51"
  • Expect on next startup that "target 1" is removed

Actual behavior

  • "target 51" is removed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kawaiipantsu kawaiipantsu

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Target History (Max hist-target cleanup, wrong order) kawaiipantsu/redjoust
1 participant
@kawaiipantsu