Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add spaniels breed support into the doggos app #4

Open
nickImbirev opened this issue May 6, 2022 · 2 comments
Open

Add spaniels breed support into the doggos app #4

nickImbirev opened this issue May 6, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@nickImbirev
Copy link
Collaborator

nickImbirev commented May 6, 2022

As a DOGGOS application user, I would like to see the dog breed 'SPANIELS' in the output report of the application.

INPUT:
Screenshot 2022-05-11 at 16 53 28

AS IS:
OUTPUT:
Screenshot 2022-05-11 at 16 49 14

TO BE:
OUTPUT:
Screenshot 2022-05-11 at 16 53 40

@nickImbirev nickImbirev added the enhancement New feature or request label May 6, 2022
nickImbirev pushed a commit that referenced this issue May 10, 2022
@nickImbirev
Copy link
Collaborator Author

nickImbirev commented May 11, 2022

Technical scenario to complete the feature request:

Please, assign this issue to you, once you start working on it, so everybody can see it.

  1. Open the installed repository and create a Git branch for the enhancement (suggested branch name: add-spaniels-support).
  2. Take a look into the program source code according to the guide.
  3. Update the source code module DOGGOS.cbl.
  • Register a new dog breed in the BREED-INDEXES structure (follow the existing breeds for the examples). Please, do not forget to update the breed index value.
  • Add a new use case into the UPDATE-ACCUMULATOR section (follow the existing breeds for the examples).
  • Add a snippet to write the new dog breed results into the output after the line 97 (follow the existing breeds for the examples).
    Note: Do not hesitate to use COBOL Control flow and COBOL LSP VSCode extensions for this change.
  1. Build the source code according to the guide.
  2. Update the test input file according to the requirements using the guide.
  3. Run the manual test with the code using the guide.
  4. (Optional): If you have issues with the understanding, why the changes are not working properly - please, debug the application using the guide.
  5. (Optional): If you feel confident about the changes, please, run the regression automated tests suite according to the guide and update the test cases with your changes.

After you did the change, please, make a git commit and push your changes into the remote repository, open a Pull Request into the main branch with your changes and ask for the code review.

@nickImbirev
Copy link
Collaborator Author

Traditional scenario

Technical scenario to complete the feature request:

Please, assign this issue to you, once you start working on it, so everybody can see it.

  1. Connect to the Endevor ISPF panel using the guide.
  2. Update the source code element DOGGOS of the COBOL type using the guide.
  • Register a new dog breed in the BREED-INDEXES structure (follow the existing breeds for the examples). Please, do not forget to update the breed index value.
  • Add a new use case into the UPDATE-ACCUMULATOR section (follow the existing breeds for the examples).
  • Add a snippet to write the new dog breed results into the output after the line 97 (follow the existing breeds for the examples).
  1. Build the source code according to the guide.
  2. Update the test input file according to the requirements using the guide.
  3. Run the manual test with the code using the guide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants