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
kernelport wants to merge 1 commit into from
Closed

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

kernelport 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

max-wittig commented Jun 30, 2020

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

JohnVillalovos commented Jan 9, 2022

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

JohnVillalovos commented Jan 13, 2022

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

JohnVillalovos commented Jan 13, 2022

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

@JohnVillalovos
Copy link
Member

JohnVillalovos commented Jan 13, 2022

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