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

Redefine probation services with model object interface #50

Merged
merged 7 commits into from
Aug 13, 2020

Conversation

sldblog
Copy link
Contributor

@sldblog sldblog commented Aug 12, 2020

What does this pull request do?

It mostly moves definitions of systems and relationships out of defineModelWithDeprecatedSyntax and into the new(ish) interface kicked off in #15.

Please review commit-by-commit (the commit messages include context).

What is the intent behind these changes?

Continuing to remove the old syntax for defining systems.

@sldblog sldblog requested a review from a team as a code owner August 12, 2020 16:21
@sldblog sldblog force-pushed the probation-services-to-new-interface branch from d6ab5ff to ee1a0f6 Compare August 12, 2020 16:22
Copy link
Contributor Author

@sldblog sldblog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clarifying notes

src/main/kotlin/model/DataPipelineTools.kt Outdated Show resolved Hide resolved
src/main/kotlin/model/DataPipelineTools.kt Outdated Show resolved Hide resolved
override fun defineModelEntities(model: Model) {
system = model.addSoftwareSystem(
"Workload Management Tool",
"(WMT) Helps probation practitioners schedule their time based on service user risk"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 This definition slightly changed to reflect that this tool is only used in the community. As a result, we can use the "probation practitioners" and "service user" terms.

src/main/kotlin/model/CourtUsers.kt Show resolved Hide resolved
src/main/kotlin/model/DataPipelineTools.kt Outdated Show resolved Hide resolved
src/main/kotlin/model/Delius.kt Outdated Show resolved Hide resolved
src/main/kotlin/model/Delius.kt Show resolved Hide resolved
@sldblog sldblog force-pushed the probation-services-to-new-interface branch from ee1a0f6 to 8b0f3c1 Compare August 13, 2020 09:09
@sldblog
Copy link
Contributor Author

sldblog commented Aug 13, 2020

♻️ rebased after previous merges

Copy link
Contributor

@spikeheap spikeheap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 again

@sldblog sldblog force-pushed the probation-services-to-new-interface branch from c949546 to e9b07f8 Compare August 13, 2020 10:25
@sldblog sldblog merged commit cbe7a9f into main Aug 13, 2020
@sldblog sldblog deleted the probation-services-to-new-interface branch August 13, 2020 10:34
@sldblog
Copy link
Contributor Author

sldblog commented Aug 13, 2020

🎩 🙇‍♂️

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

Successfully merging this pull request may close these issues.

None yet

2 participants