# Contributing but not a bug or vulnerability
**URL:** https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975
**Category:** Development
**Created:** [November 20, 2024, 12:43am UTC](https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975 "2024-11-20T00:43:53Z")
**Posts on this page:** 5
**Page:** 1
### Author:  [@yukoncornelius58](https://forums.rocket.chat/u/yukoncornelius58)
#### Post date: [November 20, 2024, 12:43am UTC](https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975/1 "2024-11-20T00:43:53Z")
Hello, I was wanting to contribute to the community and I found a couple references to a deprecated javascript substr() where it would be better to use slice(). I was going to open an issue, but it doesn’t really qualify as a “bug” or a “vulnerability.”
I was going to open this as an issue and volunteer to tackle the issue, but is there a process for these sorts of code improvements that aren’t bugs?
---
### Author:  [@reetp](https://forums.rocket.chat/u/reetp)
#### Post date: [November 20, 2024, 11:05am UTC](https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975/2 "2024-11-20T11:05:20Z")
> [@yukoncornelius58](#):
>
> I was going to open this as an issue and volunteer to tackle the issue, but is there a process for these sorts of code improvements that aren’t bugs?
Hi.
Yes. Please ask here or on open.rocket.chat whether a fix is required before wasting time on trying to fix things and opening issues that will likely be closed.
I presume this is you
>
>
> There is a deprecated use of substr() in /apps/meteor/app/autotranslate/server/g…oogleTranslate.ts that I found while doing a code review for a college class.
>
> It isn't necessarily a bug or vulnerability in the code, not a new feature, but perhaps a code improvement.
>
> The use of substr is in line 134 and 181, which I believe would be easily updated to slice(). I would volunteer to update this if the maintainers would like this to be updated.
I will ask them team but generally this sort of thing will get picked up and fixed by the team in time.
---
### Author:  [@yukoncornelius58](https://forums.rocket.chat/u/yukoncornelius58)
#### Post date: [November 23, 2024, 1:53am UTC](https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975/3 "2024-11-23T01:53:50Z")
Totally understand! Thanks for the reply!
---
### Author:  [@yukoncornelius58](https://forums.rocket.chat/u/yukoncornelius58)
#### Post date: [November 26, 2024, 2:45pm UTC](https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975/4 "2024-11-26T14:45:47Z")
Hey reetp, I was tasked with this (thanks for letting me contribute!) and submitted a PR with the fix. However, I am getting this message on the PR
- This PR is missing the ‘stat: QA assured’ label
- This PR is missing the required milestone or project
I am guessing this means it needs to be review by the internal team? Am i guessing correctly?
Also I’m failing…
**CI / deploy-preview (pull\_request)**
**CI / 🚢 Build Docker Images for Testing (alpine) (pull\_request)**
Then it would appear that the Docker test failing skips 12 tests after it fails. Do you have anywhere you’d recommend looking to find what it is causing the failure? I didn’t change anything besides substr and slice in /apps/meteor/app/autotranslate/server/googleTranslate.ts
---
### Author:  [@reetp](https://forums.rocket.chat/u/reetp)
#### Post date: [November 26, 2024, 9:17pm UTC](https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975/5 "2024-11-26T21:17:26Z")
> [@yukoncornelius58](#):
>
> Hey reetp, I was tasked with this (thanks for letting me contribute!)
No one needs permission… just do it, though it is always better to check that it is OK to work on it as often these things are taken up internally.
> [@](#):
>
> and submitted a PR with the fix. However, I am getting this message on the PR
Yup it needs the team to look. At this point you just have to be patient. It can take weeks, if not months sometimes. Rocket is not a bleeding edge project…