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

Replace deprecated String.prototype.substr() #1376

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

cabljac
Copy link
Contributor

@cabljac cabljac commented Jan 3, 2023

(reopening #928 for testing purposes)

.substr() is deprecated so we replace it with .slice() which works similarily but isn't deprecated

@cabljac cabljac marked this pull request as ready for review February 10, 2023 15:48
@cabljac cabljac added this to Under consideration in Extension Update Tracker via automation Feb 10, 2023
@cabljac cabljac moved this from Under consideration to In review in Extension Update Tracker Feb 10, 2023
.substr() is deprecated so we replace it with .slice() which works similarily but isn't deprecated

Signed-off-by: Tobias Speicher <rootcommander@gmail.com>
@cabljac cabljac added the in-review Awaiting review by FE team. label Feb 14, 2023
Extension Update Tracker automation moved this from In review to Launch approval pending Mar 6, 2023
@dackers86 dackers86 merged commit 89552c3 into next Mar 15, 2023
Extension Update Tracker automation moved this from Launch approval pending to Launched Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-review Awaiting review by FE team.
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants