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

fix: encode docname when routing (backport #17092) #17374

Merged
merged 8 commits into from Sep 7, 2022

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jul 1, 2022

This is an automatic backport of pull request #17092 done by Mergify.

(cherry picked from commit c8777c5)

# Conflicts:
#	frappe/public/js/frappe/list/list_view.js
@mergify mergify bot requested a review from a team as a code owner July 1, 2022 10:03
@mergify mergify bot requested review from phot0n and removed request for a team July 1, 2022 10:03
@mergify mergify bot added the conflicts label Jul 1, 2022
@mergify mergify bot assigned phot0n Jul 1, 2022
@phot0n
Copy link
Contributor

phot0n commented Jul 1, 2022

this will need some additional changes @surajshetty3416 - please don't merge for now :)

@ankush ankush marked this pull request as draft July 1, 2022 11:07
@codecov
Copy link

codecov bot commented Jul 1, 2022

Codecov Report

Merging #17374 (5bd1983) into develop (a35880a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##           develop   #17374    +/-   ##
=========================================
  Coverage    62.35%   62.35%            
=========================================
  Files          753      753            
  Lines        68079    67862   -217     
  Branches      5970     5968     -2     
=========================================
- Hits         42450    42318   -132     
+ Misses       22136    22041    -95     
- Partials      3493     3503    +10     
Flag Coverage Δ
server-ui 29.18% <ø> (+0.03%) ⬆️
ui-tests 49.65% <100.00%> (-0.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@stale stale bot added the inactive label Jul 8, 2022
@frappe frappe deleted a comment from stale bot Jul 8, 2022
@stale stale bot removed the inactive label Jul 8, 2022
@stale stale bot added the inactive label Jul 15, 2022
@frappe frappe deleted a comment from stale bot Jul 18, 2022
@stale stale bot removed the inactive label Jul 18, 2022
@phot0n phot0n marked this pull request as ready for review July 22, 2022 16:10
@phot0n phot0n removed the conflicts label Jul 22, 2022
@phot0n phot0n marked this pull request as draft July 22, 2022 16:27
@stale
Copy link

stale bot commented Jul 29, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@stale stale bot added the inactive label Jul 29, 2022
@stale stale bot closed this Aug 2, 2022
@mergify mergify bot deleted the mergify/bp/develop/pr-17092 branch August 2, 2022 03:02
@sagarvora sagarvora restored the mergify/bp/develop/pr-17092 branch August 2, 2022 03:29
@sagarvora sagarvora reopened this Aug 2, 2022
@stale stale bot removed the inactive label Aug 2, 2022
@stale stale bot added the inactive label Aug 9, 2022
@frappe frappe deleted a comment from stale bot Aug 9, 2022
@stale stale bot closed this Aug 14, 2022
@mergify mergify bot deleted the mergify/bp/develop/pr-17092 branch August 14, 2022 17:08
@ankush ankush restored the mergify/bp/develop/pr-17092 branch August 15, 2022 05:47
@ankush ankush reopened this Aug 15, 2022
@stale stale bot removed the inactive label Aug 15, 2022
@stale stale bot added the inactive label Aug 31, 2022
@frappe frappe deleted a comment from stale bot Aug 31, 2022
@stale stale bot removed the inactive label Sep 2, 2022
@phot0n
Copy link
Contributor

phot0n commented Sep 7, 2022

Seems to be working now:

Screen.Recording.2022-09-07.at.8.52.55.PM.mov

@phot0n phot0n marked this pull request as ready for review September 7, 2022 15:25
@phot0n phot0n added squash backport version-14-hotfix backport to version 14 and removed backport version-14-hotfix backport to version 14 labels Sep 7, 2022
@phot0n
Copy link
Contributor

phot0n commented Sep 7, 2022

realized this would need to be backported to v14-hotfix after #17537 is backported 😢

@phot0n phot0n merged commit eb4c4fb into develop Sep 7, 2022
@phot0n phot0n deleted the mergify/bp/develop/pr-17092 branch September 7, 2022 15:55
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2022
@ankush ankush added the backport version-14-hotfix backport to version 14 label Nov 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants