Skip to content

fix: allow updating of sub-pages in wikis #1080

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

fix: allow updating of sub-pages in wikis #1080

wants to merge 1 commit into from

Conversation

kernelport
Copy link
Contributor

@kernelport kernelport commented Apr 22, 2020 •
edited by JohnVillalovos
Loading

Make it possible to update subpages in wikis

Update SaveMixin to use utils._url_encode() on the ID if it is a
string.

Closes: #1079

@max-wittig
Copy link
Member

Hi @kernelport Sorry for the delay! Would you mind to add a quick unit test for this and use conventional changelog for the commit message

@nejch nejch mentioned this pull request Mar 2, 2021
Closed
@JohnVillalovos JohnVillalovos changed the title make it possible to update subpages in wikis fix: allow updating of sub-pages in wikis Jan 9, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jan 9, 2022 •
edited
Loading

Codecov Report

Merging #1080 (535636f) into main (ac81272) will increase coverage by 0.00%.
The diff coverage is 80.00%.

@@           Coverage Diff           @@
##             main    #1080   +/-   ##
=======================================
  Coverage   92.10%   92.11%           
=======================================
  Files          76       76           
  Lines        4801     4804    +3     
=======================================
+ Hits         4422     4425    +3     
  Misses        379      379
Flag Coverage Δ
cli_func_v4 81.30% <20.00%> (-0.06%) ⬇️
py_func_v4 80.18% <80.00%> (+0.01%) ⬆️
unit 83.05% <80.00%> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
gitlab/mixins.py 91.60% <80.00%> (+0.06%) ⬆️

@kernelport @JohnVillalovos
fix: allow updating of sub-pages in wikis
535636f 
Make it possible to update subpages in wikis

Update SaveMixin to use utils._url_encode() on the ID if it is a
string.

Closes: python-gitlab#1079
@JohnVillalovos JohnVillalovos requested a review from nejch January 9, 2022 01:38
@JohnVillalovos JohnVillalovos marked this pull request as draft January 9, 2022 01:38
@JohnVillalovos
Copy link
Member

So I'm not sure if this is the way to go. I was thinking maybe we should add a get_url_encoded_id method around here: https://github.com/python-gitlab/python-gitlab/blob/main/gitlab/base.py#L205-L209

@JohnVillalovos
Copy link
Member

Closing this as done by #1819

@nejch
Copy link
Member

nejch commented Jan 13, 2022

@JohnVillalovos I think there might be parts of this that are useful for other issues, see #1081. Notably returning the save results so that moved subpages can be edited. But agreed, the url-encoding is finished.

@JohnVillalovos
Copy link
Member

I'll work on a new PR to return the dictionary from save()

@JohnVillalovos
Copy link
Member

I created #1832 for the save() result.

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

@nejch nejch Awaiting requested review from nejch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

project.wikis.create with subpages results in 404 Wiki Page Not Found
5 participants
@kernelport @max-wittig @codecov-commenter @JohnVillalovos @nejch