Skip to content

Conversation

@CommanderRoot
Copy link
Contributor

String.prototype.substr() is deprecated so we replace it with String.prototype.slice() which works similarily but isn't deprecated.
.substr() probably isn't going away anytime soon but the change is trivial so it doesn't hurt to do it.

@artonge
Copy link
Collaborator

artonge commented Mar 24, 2022

Not quite the same, but in this case, it works :)

@nickvergessen nickvergessen removed their request for review April 19, 2022 13:30
@nickvergessen
Copy link
Member

@artonge feel free to merge

@nickvergessen
Copy link
Member

cypress Expected — Waiting for status to be reported

This is a bit suspicious thou :P

@artonge
Copy link
Collaborator

artonge commented Apr 19, 2022

@artonge feel free to merge

I can't, need more rights for that :'( (as CI is not fully green)

Edit: I can check why cypress is hanging

@CommanderRoot
Copy link
Contributor Author

I have rebased the PR on the current master to hopefully restart the workflows and maybe make them work this time.

@nickvergessen
Copy link
Member

Can you rebase on top of #789

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

Signed-off-by: Tobias Speicher <[email protected]>
@artonge artonge merged commit 08837c0 into nextcloud:master Apr 20, 2022
@artonge
Copy link
Collaborator

artonge commented Apr 20, 2022

Thanks for this @CommanderRoot !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants