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

Skip to content

Conversation

@brodock
Copy link
Contributor

@brodock brodock commented Jan 30, 2017

You can use int based ID to identify a project, or you can use NAMESPACE/REPO according to the API documentation: https://docs.gitlab.com/ce/api/projects.html#get-single-project

The advantage of using NAMESPACE/REPO everywhere, is that you don't need to first load the repository to get the numeric ID so you can use it on other places.

The gem itself already allows the use of the "literal" identification, but there is one check in the issues getter that prevents it's use. This PR fixes that.

cc @NARKOZ @connorshea @asedge

@brodock brodock changed the title Allow issues to use NAMESPACE/REPO identifier WIP: Allow issues to use NAMESPACE/REPO identifier Jan 30, 2017
@brodock brodock changed the title WIP: Allow issues to use NAMESPACE/REPO identifier Allow issues to use NAMESPACE/REPO identifier Jan 30, 2017
@asedge
Copy link
Collaborator

asedge commented Feb 14, 2017

Thanks for the contribution but I think this was already covered in #253!

@asedge asedge closed this Feb 14, 2017
@bergholdt
Copy link
Contributor

@asedge we still need this one.
Cifrene Issue code was checking for integer value

@bergholdt
Copy link
Contributor

Cifrene = current
iPhone autocorrect was on wrong language

@asedge asedge reopened this Feb 14, 2017
@asedge asedge merged commit bd85ddf into NARKOZ:master Feb 14, 2017
@asedge
Copy link
Collaborator

asedge commented Feb 14, 2017

OK, merged. Thanks!

andrew added a commit to librariesio/gitlab that referenced this pull request Apr 11, 2017
* upstream/master:
  release 4.0.0
  Move .file_contents to RepositoryFiles module, matching GitlabHQ API doc.  Also update route so it actually works with APIv4.
  Fix .get_file for new APIv4 route
  Update gitignore
  Add ability to get all issues a merge_request will close (NARKOZ#271)
  Update CHANGELOG, add some missing entries.
  Block/unblock user now uses POST instead of PUT.  Fixes NARKOZ#262.
  Adds ability to create commits (NARKOZ#264)
  Rename "git hook" to "push rule".  Closes NARKOZ#233.
  Change project fork endpoint for v4 API. Closes NARKOZ#236.
  Add v4-endpoint to fetch all merge_requests for a milestone (NARKOZ#265)
  Fixes NARKOZ#222, using deploy_keys endpoint (NARKOZ#261)
  (Un)subscribe to issues and merge_requests (NARKOZ#260)
  Update merge_request endpoints to new merge_requests endpoints (NARKOZ#256)
  Fix failing test that was called with a pre-urlencoded string.
  Allow issues to use NAMESPACE/REPO identifier (NARKOZ#255)
  Feature/230 note that project id can also be a string (NARKOZ#253)
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