u/pc/rhodecode-enterprise-ce-fork-pc Files · docs/collaboration/update-pull-request.rst

auth: don't break hashing in case of user with empty password....

auth: don't break hashing in case of user with empty password. In some cases such as LDAP user created via external scripts users might set the passwords to empty. The hashing uses the md5(password_hash) to store reference to detect password changes and forbid using the same password. In case of pure LDAP users this is not valid, and we shouldn't raise Errors in such case. This change makes it work for empty passwords now.

marcink - - Load All Authors

File last commit:

r1:854a839a default


                r2203:8a18c3c3

default

Download file

             update-pull-request.rst
        
                    47 lines
            
             | 1.4 KiB
            
                | text/x-rst
            
             |
                RstLexer
            
             / docs / collaboration / update-pull-request.rst
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        marcink
    
project: added all source files and assets

              r1
            
      .. _update-requests-ref:

      Update a |pr|

      -------------

      .. important::

         Updating a |pr| invalidates all previous reviews, and the status set by

         each reviewer. This means all reviewers need to review and approve the

         |pr| again.

      If you have made changes on the fork or branch from where a |pr| originates,

      you can update the |pr|, using the following steps:

      1. Commit and push your changes to the |RCE| server.

      2. Open the |pr| you wish to update, and select the :guilabel:`Update commits`

         button.

      .. image:: ../images/update-pr-button.png

         :scale: 50 %

         :alt: Update Pull Request

         :align: Center

      Once updated the following commit message will be automatically added to the

      |pr|.

      :guilabel:`Auto status change to "Under Review", pull request updated with 1 added, 0 removed commits.`

      3. Review the new commit messages and changes as before,

      4. Set the review status and click :guilabel:`Comment`

      Inline Comments

      ^^^^^^^^^^^^^^^

      When updating a |pr|, inline comments can move around depending on which line

      they are attached:

      * If the line content changes but remains in place, the inline comment will

        remain where it was originally.

      * If the line to which the inline comment is attached moves, the comment will

        move with it.

      * If the diff changes substantially and the line to which the comment was

        attached is no longer in the diff, the inline comment will be hidden but

        with an icon to display it if necessary.

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

marcink project: added all source files and assets	r1	.. _update-requests-ref:

		Update a \|pr\|
		-------------

		.. important::

		Updating a \|pr\| invalidates all previous reviews, and the status set by
		each reviewer. This means all reviewers need to review and approve the
		\|pr\| again.

		If you have made changes on the fork or branch from where a \|pr\| originates,
		you can update the \|pr\|, using the following steps:

		1. Commit and push your changes to the \|RCE\| server.
		2. Open the \|pr\| you wish to update, and select the :guilabel:`Update commits`
		button.

		.. image:: ../images/update-pr-button.png
		:scale: 50 %
		:alt: Update Pull Request
		:align: Center


		Once updated the following commit message will be automatically added to the
		\|pr\|.

		:guilabel:`Auto status change to "Under Review", pull request updated with 1 added, 0 removed commits.`

		3. Review the new commit messages and changes as before,
		4. Set the review status and click :guilabel:`Comment`

		Inline Comments
		^^^^^^^^^^^^^^^

		When updating a \|pr\|, inline comments can move around depending on which line
		they are attached:

		* If the line content changes but remains in place, the inline comment will
		remain where it was originally.
		* If the line to which the inline comment is attached moves, the comment will
		move with it.
		* If the diff changes substantially and the line to which the comment was
		attached is no longer in the diff, the inline comment will be hidden but
		with an icon to display it if necessary.