-
Notifications
You must be signed in to change notification settings - Fork 17
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
N21-1494 Contextual info outdated CTL #4572
Conversation
apps/server/src/modules/tool/common/service/common-tool.service.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/domain/tool-configuration-status.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/service/common-tool.service.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/service/common-tool.service.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/service/common-tool.service.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/context-external-tool/service/tool-reference.service.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/context-external-tool/service/tool-version-service.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/domain/tool-configuration-status.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/school-external-tool/controller/api-test/tool-school.api.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/school-external-tool/service/school-external-tool.service.ts
Outdated
Show resolved
Hide resolved
isOutdatedOnScopeContext: false, | ||
isOutdatedOnScopeSchool: false, | ||
}); | ||
|
||
try { |
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.
You could try to use a Boolean in the validation functions if it's easy possible
apps/server/src/modules/tool/school-external-tool/controller/api-test/tool-school.api.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/school-external-tool/service/school-external-tool.service.ts
Outdated
Show resolved
Hide resolved
isOutdatedOnScopeSchool: true, | ||
}); | ||
|
||
const latestConfigurationStatus: ToolConfigurationStatus = new ToolConfigurationStatus({ |
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.
Do you really need two instances ? I could imagine a static function or class which you could use here
apps/server/src/modules/tool/tool-launch/service/tool-launch.service.ts
Outdated
Show resolved
Hide resolved
apps/server/src/shared/testing/factory/domainobject/tool/tool-configuration-status.factory.ts
Outdated
Show resolved
Hide resolved
apps/server/src/shared/testing/factory/tool-configuration-status-response.factory.ts
Outdated
Show resolved
Hide resolved
...er/src/modules/tool/school-external-tool/mapper/school-external-tool-response.mapper.spec.ts
Outdated
Show resolved
Hide resolved
...server/src/modules/tool/school-external-tool/controller/dto/school-external-tool.response.ts
Outdated
Show resolved
Hide resolved
.../src/modules/tool/context-external-tool/controller/dto/tool-configuration-status.response.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/domain/tool-configuration-status.ts
Outdated
Show resolved
Hide resolved
# Conflicts: # apps/server/src/modules/tool/context-external-tool/controller/api-test/tool-reference.api.spec.ts # apps/server/src/modules/tool/context-external-tool/service/tool-reference.service.ts # apps/server/src/modules/tool/context-external-tool/uc/tool-reference.uc.spec.ts # apps/server/src/modules/tool/tool-launch/service/tool-launch.service.ts
# Conflicts: # apps/server/src/modules/tool/common/service/common-tool.service.spec.ts # apps/server/src/modules/tool/common/service/common-tool.service.ts # apps/server/src/shared/testing/factory/index.ts # backup/setup/external-tools.json # backup/setup/school-external-tools.json
apps/server/src/modules/tool/school-external-tool/service/school-external-tool.service.spec.ts
Outdated
Show resolved
Hide resolved
...er/src/modules/tool/school-external-tool/mapper/school-external-tool-response.mapper.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/school-external-tool/controller/api-test/tool-school.api.spec.ts
Outdated
Show resolved
Hide resolved
...server/src/modules/tool/context-external-tool/controller/api-test/tool-reference.api.spec.ts
Outdated
Show resolved
Hide resolved
apps/server/src/modules/tool/common/mapper/tool-status-response.mapper.ts
Outdated
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! |
Description
This PR handles the generation of infos about ctl tools in the tool board element and in courses/rooms. Specific outdated message will be shown for teacher and students.
Links to Tickets or other pull requests
https://ticketsystem.dbildungscloud.de/browse/N21-1494
hpi-schul-cloud/nuxt-client#2931
Changes
Datasecurity
Deployment
New Repos, NPM pakages or vendor scripts
Approval for review
generate-client:server
was executed in vue frontend and changes were tested and put in a PR with the same branch name.