Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@akashdotsrivastava
Copy link
Collaborator

@akashdotsrivastava akashdotsrivastava commented Nov 21, 2018

Wikis API

Gitlab.wikis(project)
Gitlab.wiki(project, slug)
Gitlab.create_wiki(project, title, content, options = {})
Gitlab.update_wiki(project, slug, options)
Gitlab.delete_wiki(project, slug)

Closes #437

@NARKOZ NARKOZ merged commit bcd3e95 into NARKOZ:master Nov 21, 2018
@NARKOZ
Copy link
Owner

NARKOZ commented Nov 21, 2018

Thank you ❤️

#
# @example
# Gitlab.update_wiki(6, 'home', { title: 'New title' })
# Gitlab.update_wiki(6, 'home', { title: 'New title', message: 'New Message', format: 'rdoc' })
Copy link
Contributor

Choose a reason for hiding this comment

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

message?

Copy link
Owner

Choose a reason for hiding this comment

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

PR is welcome

#
# @example
# Gitlab.create_wiki(3, 'Some Content', 'Some Title')
# Gitlab.create_wiki(3, 'Some Content', 'Some Title', { format: 'rdoc' })
Copy link
Contributor

Choose a reason for hiding this comment

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

The content and title need to be switched here, and in the parameters below

#
# @param [Integer, String] project The ID or name of a project.
# @param [Hash] options A customizable set of options.
# @option options [String] :with_content(optional) Include pages content
Copy link
Contributor

Choose a reason for hiding this comment

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

Is :with_content supposed to be a symbol?

Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like it is, but the other options in this file aren't symbols, that should probably be fixed :)

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