Skip to content
Snippets Groups Projects

Fixed UX Guide link on README.md

Closed Fixed UX Guide link on README.md
kartikey1/gitlab-ce:fix_ux_guide_link_READMEmd into master
Closed username-removed-508252 requested to merge kartikey1/gitlab-ce:fix_ux_guide_link_READMEmd into master

What does this MR do?

Fixed UX Guide link on README.md

(briefly describe what this MR is about) With the following commit UX Guide link on README.md was added but pointed at a wrong link. Ref commit: 9d0dc681

Moving docs to a new location?

See the guidelines: http://docs.gitlab.com/ce/development/doc_styleguide.html#changing-document-location

  • Make sure the old link is not removed and has its contents replaced with a link to the new location.
  • Make sure internal links pointing to the document in question are not broken.
  • Search and replace any links referring to old docs in GitLab Rails app, specifically under the app/views/ directory.
  • If working on CE, submit an MR to EE with the changes as well.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • username-removed-508252 Marked the task Make sure the old link is not removed and has its contents replaced with a link to the new location. as completed

    Marked the task Make sure the old link is not removed and has its contents replaced with a link to the new location. as completed

  • username-removed-508252 Marked the task Make sure the old link is not removed and has its contents replaced with a link to the new location. as incomplete

    Marked the task Make sure the old link is not removed and has its contents replaced with a link to the new location. as incomplete

  • username-removed-508252 Marked the task Make sure internal links pointing to the document in question are not broken. as completed

    Marked the task Make sure internal links pointing to the document in question are not broken. as completed

  • username-removed-508252 Marked the task Search and replace any links referring to old docs in GitLab Rails app, specifically under the app/views/ directory. as completed

    Marked the task Search and replace any links referring to old docs in GitLab Rails app, specifically under the app/views/ directory. as completed

  • username-removed-508252 Marked the task If working on CE, submit an MR to EE with the changes as well. as completed

    Marked the task If working on CE, submit an MR to EE with the changes as well. as completed

  • username-removed-508252 Marked the task Make sure internal links pointing to the document in question are not broken. as incomplete

    Marked the task Make sure internal links pointing to the document in question are not broken. as incomplete

  • username-removed-508252 Marked the task Search and replace any links referring to old docs in GitLab Rails app, specifically under the app/views/ directory. as incomplete

    Marked the task Search and replace any links referring to old docs in GitLab Rails app, specifically under the app/views/ directory. as incomplete

  • username-removed-508252 Marked the task If working on CE, submit an MR to EE with the changes as well. as incomplete

    Marked the task If working on CE, submit an MR to EE with the changes as well. as incomplete

  • username-removed-626804 Added ~15078 ~12980 labels

    Added ~15078 ~12980 labels

  • username-removed-508252
    username-removed-508252 @kartikey1 ·
    Author Contributor

    @pedroms anything I will have to do here? Make the tests pass or something? Because it's a very small bug.

  • username-removed-626804 closed

    closed

  • username-removed-626804
    username-removed-626804 @pedroms ·
    Developer

    @kartikey1 this wasn't merged because the tests weren't passing. However, this change was implemented by https://gitlab.com/gitlab-org/gitlab-ce/commit/ac761e450919b7a2d2dd76e08afe2a877e878e61 in the meantime. Thank you anyway 👍

Please register or sign in to reply