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.

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

Deprecated use of substr in googleTranslate.ts #34007

Open
jpulattie opened this issue Nov 20, 2024 · 4 comments
Open

Deprecated use of substr in googleTranslate.ts #34007

jpulattie opened this issue Nov 20, 2024 · 4 comments
Labels
Tasked Added to the internal issue tracking

Comments

@jpulattie
Copy link

There is a deprecated use of substr() in /apps/meteor/app/autotranslate/server/googleTranslate.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.

@reetp
Copy link

reetp commented Nov 20, 2024

I think this is you.

https://forums.rocket.chat/t/contributing-but-not-a-bug-or-vulnerability/20975

I will refer this to the team but it will probably be handled internally. Someone will let you know.

@reetp reetp added the Tasked Added to the internal issue tracking label Nov 20, 2024
@thepiyush-303
Copy link
Contributor

Is anyone currently working on this issue? If not, may I take it?

@reetp
Copy link

reetp commented Nov 21, 2024

Is anyone currently working on this issue? If not, may I take it?

I will refer this to the team but it will probably be handled internally. Someone will let you know.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

@casalsgh
Copy link
Contributor

If you want to give it a go with a PR, please do jpulattie. Will be nice =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tasked Added to the internal issue tracking
Projects
None yet
Development

No branches or pull requests

4 participants