Skip to content

Conversation

@daniel-manta-hee
Copy link
Contributor

@daniel-manta-hee daniel-manta-hee commented Jan 26, 2022

JIRA link

https://hee-dls.atlassian.net/browse/DLSV2-477

Description

When listing competency resources, also call learning hub api to ensure it has up-to-date information in the database about the resource. If the api is unavailable, use database information.

Other related data like e.g. parameters are always stored in database, so api is used to override some resource fields likely to change as name or description.

Screenshots

Switch application, go to Frameworks, Framework Structure and click on "Manage resource signposting".

Competency resource links with and without parameter

image

Learning Hub api is unavailable

image

Unmatched resource warning

image


Developer checks

  • Checked that Learning Hub api "is unavailable" warning message is in place when no internet connection.
  • Checked that data retrieved in Swagger overrides database data for resources.
  • Checked that unmatched resource is in place: add a record in LearningResourceReferences for ResourceRefID number 66666 that is unmatched in Swagger when calling bulk method
  • Checked that Add Resource button is disabled when Learning Hub api is unavailable.

@daniel-manta-hee daniel-manta-hee changed the title Develop/features/dlsv2 477 lh api to populate edit competency learning resources view DLSV2-477 Learning Hub api to populate EditCompetencyLearningResources view Jan 26, 2022
…gue name in round parentesis when listing resource links from EditCompetencyLearningResources view
Copy link
Contributor

@kevwhitt-hee kevwhitt-hee left a comment

Choose a reason for hiding this comment

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

All good.

@kevwhitt-hee kevwhitt-hee merged commit 099aa32 into master Jan 27, 2022
@kevwhitt-hee kevwhitt-hee deleted the Develop/Features/DLSV2-477-LhApiToPopulateEditCompetencyLearningResourcesView branch January 27, 2022 13:28
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.

3 participants