Skip to content

Revert "DRY up the code for the backTo param in routes" - #30142

Closed
thienlnam wants to merge 1 commit into
mainfrom
revert-29813-tgolen-routes-backto
Closed

thienlnam wants to merge 1 commit into
mainfrom
revert-29813-tgolen-routes-backto

Conversation

@thienlnam

Copy link
Copy Markdown
Contributor

Reverts #29813

@thienlnam thienlnam self-assigned this Oct 22, 2023
@thienlnam
thienlnam marked this pull request as ready for review October 22, 2023 01:28
@thienlnam
thienlnam requested a review from a team as a code owner October 22, 2023 01:28
@thienlnam

Copy link
Copy Markdown
Contributor Author

Straight revert

@melvin-bot
melvin-bot Bot removed the request for review from a team October 22, 2023 01:28
@melvin-bot

melvin-bot Bot commented Oct 22, 2023

Copy link
Copy Markdown

@Julesssss Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@melvin-bot
melvin-bot Bot requested a review from Julesssss October 22, 2023 01:28

@luacmartins luacmartins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should go with this fix instead of reverting

@Julesssss

Copy link
Copy Markdown
Contributor

I think we should go with #30144 instead of reverting

Seems like we did that. Closing

@Julesssss Julesssss closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants