-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
Added a toggle to filter out ammo types you can't buy based on your trader levels #930
Conversation
.deploy to development |
|
@Razzmatazzz π |
.deploy to development |
|
.deploy to development |
Deployment Triggered πBlightbuster, started a branch deployment to development You can watch the progress here π
|
Deployment Results βBlightbuster successfully deployed branch Show Results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the translation strings to src/translations/en/translation.json
please.
For some reason I cant push my commit to the PR.
"Trader Ammo": "Trader Ammo",
"Only show ammo available from traders on your settings": "Only show ammo available from traders on your settings",
for sure! on it |
.deploy to development |
Deployment Triggered πBlightbuster, started a branch deployment to development You can watch the progress here π
|
Deployment Results βBlightbuster successfully deployed branch Show Results |
.deploy |
Deployment Triggered πBlightbuster, started a branch deployment to production You can watch the progress here π
|
Deployment Results βBlightbuster successfully deployed branch
|
Many thanks for the PR β€οΈ |
[Trader Ammo Filter]
I added a toggle below the graph that filters out all ammo types that the user can't buy.
Description ποΈ
I added a toggle below the graph that filters out all ammo types that the user can't buy. It should work with both the API from the tracker as well as the settings page. It should cover the use case for the issue but I'm open to feedback if anyone has suggestions. I made sure it updates when the user changes their settings on the page but I haven't tested if I updated well if the tarkovtracker updates.
I also spotted a typo in the contributing.md when I was making sure I was following all the rules. It was just one word so I didn't think it warranted an entire Pull request. But I gave it a separate commit in case you guys wanted to drop it.
Examples πΈ
This is for level 1 traders

Related Issues π
(#278)
Expand for Help
You need additional help and want to chat in real-time? Join our community Discord