Differences

This shows you the differences between two versions of the page.

--- newdevs:git:create [2019/07/30 09:30] – [3. Make Your Changes] tmccanna
+++ newdevs:git:create [2024/03/12 15:10] (current) – tmccanna
@@ Line 1: / Line 1: @@
+**New Developers Working Group**
 ====== Git for Windows ======
@@ Line 9: / Line 10: @@
   - Open Git Bash
   - Navigate into your Evergreen repository
-  - If the master branch isn't currently checked out, check it out: git checkout master
+  - If the main branch isn't currently checked out, check it out: git checkout main
   - Before making any changes, make sure your local files are up to date with latest changes:
 <code bash>
-git pull origin master
+git pull origin main
 </code>
@@ Line 22: / Line 23: @@
 <code bash>
+git checkout -b lp123456_launchpad_bug_description
+</code>
+Note: In most cases you will want to create a branch based off of main. You can do this either by a) specifying main on the command line:
+<code bash>
+git checkout -b lp123456_launchpad_bug_description main
+</code>
+Or, b) you can switch to main first and then create the new branch. This also gives you the opportunity to make sure your local copy of main is updated first:
+<code bash>
+git checkout main
+git pull origin main
 git checkout -b lp123456_launchpad_bug_description
 </code>
@@ Line 33: / Line 48: @@
     * To stage //all// changed files, use: git add -A
     * If you accidentally stage a file you didn't mean to, you can unstage it: git reset HEAD myfilename.txt
+  - If you are adding a new feature (as opposed to a bug fix), you will also need to create a release note file. Look in the RELEASE_NOTES_NEXT folder for a template and examples:
+    * https://git.evergreen-ils.org/?p=Evergreen.git;a=tree;f=docs/RELEASE_NOTES_NEXT
   - Commit the staged changes: **git commit -s**
     * Note: The -s portion adds the sign-off line for you.
-  - When your text editor opens to write the commit message, the first line should be the launchpad number and name, followed by the description, followed by your sign off. For example:
+  - When your text editor opens to write the commit message, the first line should be the launchpad number and name, followed by the description (with test instructions if relevant), then a brief release note line (written in present tense), and then followed by your sign off. For example:
   LP#1406387 Fix for Holds Placement Advanced Options
@@ Line 44: / Line 61: @@
   empty, regardless of whether the radio input for the
   hold was specified for a patron or the staff member.
+  To reproduce the problem:
+  [1] Open a patron's account and start the process to place a hold.
+  [2] On the hold placement screen, click Advanced Hold options.
+  [3] You will see that the staff barcode is filled in rather than the patron.
+  To test the fix:
+  [1] Open a patron's account and start the process to place a hold.
+  [2] On the hold placement screen, click Advanced Hold options.
+  [3] Verify that the patron barcode is filled in.
+  [4] Test placing a hold directly from the catalog and confirm that the
+  staff or patron barcode is carried over from the main hold page.
+  Release-note: Fixes issue where staff barcode populated the staff client advanced holds screen.
   Signed-off-by:  Jane Hacker <jhacker@example.org>