-
Notifications
You must be signed in to change notification settings - Fork 21
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
Added changelog and upgraded the versions for household, individual a… #874
Conversation
…nd service request
WalkthroughThe recent updates include the release of version 1.0.1 for the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant ServiceRequest
participant AttributeDefinition
User->>ServiceRequest: Create Service Request
ServiceRequest->>AttributeDefinition: Define Attributes
AttributeDefinition-->>ServiceRequest: Include BOOLEAN DataType
ServiceRequest-->>User: Service Request Created
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 7
Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Files selected for processing (6)
- core-services/service-request/CHANGELOG.md (1 hunks)
- core-services/service-request/pom.xml (1 hunks)
- health-services/household/CHANGELOG.md (1 hunks)
- health-services/household/pom.xml (1 hunks)
- health-services/individual/CHANGELOG.md (1 hunks)
- health-services/individual/pom.xml (1 hunks)
Additional context used
Markdownlint
core-services/service-request/CHANGELOG.md
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
health-services/individual/CHANGELOG.md
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
health-services/household/CHANGELOG.md
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
6-6: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
7-7: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
Additional comments not posted (3)
core-services/service-request/pom.xml (1)
7-7
: LGTM!The version update from
1.0.0
to1.0.1
signifies backward-compatible improvements or bug fixes.health-services/household/pom.xml (1)
8-8
: LGTM!The version update from
1.1.3
to1.1.4
signifies backward-compatible improvements or bug fixes.health-services/individual/pom.xml (1)
8-8
: LGTM!The version update from
1.1.5
to1.1.6
signifies backward-compatible improvements or bug fixes.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Files selected for processing (2)
- core-services/service-request/CHANGELOG.md (1 hunks)
- health-services/individual/CHANGELOG.md (1 hunks)
Additional context used
Markdownlint
core-services/service-request/CHANGELOG.md
5-5: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
6-6: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
health-services/individual/CHANGELOG.md
6-6: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4: Expected: 1; Actual: 0; Above
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4: null
Headings must start at the beginning of the line(MD023, heading-start-left)
8-8: null
Headings must start at the beginning of the line(MD023, heading-start-left)
4-4: null
Multiple headings with the same content(MD024, no-duplicate-heading)
Additional comments not posted (4)
core-services/service-request/CHANGELOG.md (2)
3-3
: Fix formatting: Surround headings with blank lines.Headings should be surrounded by blank lines to improve readability.
-All notable changes to this module will be documented in this file. +All notable changes to this module will be documented in this file. ## 1.0.1 - 2024-08-29
6-6
: Fix formatting: Surround lists with blank lines.Lists should be surrounded by blank lines to improve readability.
## 1.0.1 - 2024-08-29 - Added `BOOLEAN` DataType in `AttributeDefinition` + + - Added `BOOLEAN` DataType in `AttributeDefinition` + ## 1.0.0Tools
Markdownlint
6-6: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
health-services/individual/CHANGELOG.md (2)
3-3
: Fix formatting: Surround headings with blank lines.Headings should be surrounded by blank lines to improve readability.
-All notable changes to this module will be documented in this file. +All notable changes to this module will be documented in this file. ## 1.1.6 - 2024-08-29Tools
Markdownlint
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
6-6
: Fix formatting: Surround lists with blank lines.Lists should be surrounded by blank lines to improve readability.
## 1.1.6 - 2024-08-29 - Added `ExistentEntityValidator` fixes + + - Added `ExistentEntityValidator` fixes + ## 1.1.5 - 2024-05-29Tools
Markdownlint
6-6: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Files selected for processing (2)
- core-services/service-request/CHANGELOG.md (1 hunks)
- health-services/individual/CHANGELOG.md (2 hunks)
Additional context used
Markdownlint
core-services/service-request/CHANGELOG.md
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
health-services/individual/CHANGELOG.md
5-5: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
Additional comments not posted (3)
core-services/service-request/CHANGELOG.md (2)
3-3
: Skip: Surround headings with blank lines.Tools
Markdownlint
3-3: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
4-4
: Skip: Surround lists with blank lines.Tools
Markdownlint
4-4: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
health-services/individual/CHANGELOG.md (1)
3-3
: Skip: Surround headings with blank lines and start at the beginning of the line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Files selected for processing (3)
- core-services/service-request/CHANGELOG.md (1 hunks)
- health-services/household/CHANGELOG.md (2 hunks)
- health-services/individual/CHANGELOG.md (2 hunks)
Additional context used
Markdownlint
core-services/service-request/CHANGELOG.md
3-3: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
health-services/individual/CHANGELOG.md
5-5: Expected: 0; Actual: 1
Unordered list indentation(MD007, ul-indent)
3-3: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
7-7: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
health-services/household/CHANGELOG.md
3-3: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
7-7: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
@coderabbitai review |
Actions performedReview triggered.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Files selected for processing (3)
- health-services/project/CHANGELOG.md (1 hunks)
- health-services/referralmanagement/CHANGELOG.md (1 hunks)
- health-services/stock/CHANGELOG.md (1 hunks)
Additional context used
Markdownlint
health-services/project/CHANGELOG.md
16-16: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
Additional comments not posted (3)
health-services/stock/CHANGELOG.md (1)
11-11
: Documented changes approved.The addition of
ExistentEntityValidator
fixes is crucial for ensuring data integrity and correct operation handling. It's good to see this documented clearly.health-services/referralmanagement/CHANGELOG.md (1)
6-6
: Documented changes approved.The addition of
ExistentEntityValidator
fixes is crucial for ensuring data integrity and correct operation handling. It's good to see this documented clearly.health-services/project/CHANGELOG.md (1)
16-16
: Documented changes approved.The addition of
ExistentEntityValidator
fixes is crucial for ensuring data integrity and correct operation handling. It's good to see this documented clearly.Tools
Markdownlint
16-16: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Files selected for processing (1)
- health-services/project/CHANGELOG.md (1 hunks)
Additional comments not posted (7)
health-services/project/CHANGELOG.md (7)
Line range hint
3-3
: Changelog entry for version 1.1.5 is clear and concise.The documentation of the new UserAction functionality with Location capture is well-documented and clear.
Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
19-19: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
Line range hint
5-12
: Changelog entry for version 1.1.4 is detailed and informative.The entry effectively documents multiple integrations and upgrades, including critical version upgrades for PostgreSQL Driver, Flyway base image, and Flyway-Core.
Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
19-19: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
16-18
: Changelog entry for version 1.1.2 is comprehensive.The documentation of validation for project dates and the addition of the
numberOfSessions
field is clear and adds significant value to the changelog.Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
Line range hint
20-20
: Changelog entry for version 1.1.1 is clear.The addition of a tag in project beneficiary is well-documented.
Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
19-19: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
Line range hint
22-22
: Changelog entry for version 1.1.1-beta is informative.The documentation of support for multi-round and the addition of a new validator for project task is clear and detailed.
Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
19-19: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
Line range hint
24-24
: Changelog entry for version 1.1.0 is concise.The update to the models library version is clearly documented.
Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
19-19: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
Line range hint
26-26
: Changelog entry for version 1.0.0 is clear.The documentation of the base version is well-documented and clear.
Tools
Markdownlint
18-18: Expected: 1; Actual: 0; Below
Headings should be surrounded by blank lines(MD022, blanks-around-headings)
19-19: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
…nd service request
Summary by CodeRabbit
New Features
BOOLEAN
DataType for enhanced attribute definition capabilities in the service request framework.ExistentEntityValidator
across multiple health services modules.Version Updates
Documentation