Closed Razzmatazzz closed 5 months ago
If you are new, please check out the trimmed down summary of our deployment process below:
👀 Observe the CI jobs and tests to ensure they are passing
✔️ Obtain an approval/review on this pull request
🚀 Deploy your pull request to the development environment with .deploy to development
🚀 Deploy your pull request to the production environment with .deploy
If anything goes wrong, rollback with
.deploy main
🎉 Merge!
Note: If you have a larger change and want to block deployments, you can run
.lock --reason <reason>
to lock all other deployments (remove with.unlock
)
You can view the branch deploy usage guide for additional information
.deploy
Shebuka, started a branch deployment to production
You can watch the progress here 🔗
Branch:
sell-to-trader-total
Shebuka successfully deployed branch sell-to-trader-total
to production
Environment URL: tarkov.dev
When a preset item contained multiple of an item, the sell to trader column was only including the value of one of each item in the total:![image](https://github.com/the-hideout/tarkov-dev/assets/35779878/66e703b9-ce88-49ec-bb45-39a53f605c16)
Now it shows the correct total including the count of all items:![image](https://github.com/the-hideout/tarkov-dev/assets/35779878/3bcd7a68-4d83-4e01-bd99-3c8249240754)
Also fixes weird formatting issues for the item page barters filter controls:
![image](https://github.com/the-hideout/tarkov-dev/assets/35779878/c5a8af09-0356-487b-a0e8-43c555f956de)
Also makes the last string capable of translation listed in #364.
Resolves #364