✨ [Frontend] Listen to openFunction iframe's message

[odeimaiz]

What do these changes do?

The MetaModeling service might want to pop up some Function (template) information.

In this PR, the iframe handler listens to messages with the following structure:

{
  "type": "openFunction",
  "message": {
    "functionId": {function_id}
  }
}

Related issue/s

• closes Cannot edit pricing unit in frontend #7742

How to test

Dev-ops

[Copilot]

Pull Request Overview

This PR introduces a new "openFunction" message handler in the PersistentIframe widget to open function/template details and adds a corresponding fetchTemplate method in the Templates store.

• Added a new case "openFunction" in PersistentIframe.js to handle iframe messages.
• Implemented fetchTemplate in Templates.js to retrieve function/template data via an API call.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
services/static-webserver/client/source/class/osparc/widget/PersistentIframe.js	Introduces the "openFunction" case to fetch template data and display resource details.
services/static-webserver/client/source/class/osparc/store/Templates.js	Adds a fetchTemplate method to perform an API call (using the "studies" resource) for retrieving template data.

Comments suppressed due to low confidence (2)

services/static-webserver/client/source/class/osparc/widget/PersistentIframe.js:334

• [nitpick] The variable is named 'templateId' even though it is sourced from 'functionId'. Consider renaming it to 'functionId' or adjusting the field reference for clarity and consistency.

const templateId = data["message"]["functionId"];

services/static-webserver/client/source/class/osparc/store/Templates.js:52

• [nitpick] The method 'fetchTemplate' is using the 'studies' resource endpoint, which might be inconsistent with the intended purpose of fetching template data. Verify that the correct resource endpoint is used.

return osparc.data.Resources.fetch("studies", "getOne", params)

[odeimaiz]

@Mergifyio queue

[mergify]

queue

🛑 The pull request has been synchronized by a user

[pcrespov]

👀

[mergify]

This pull request has been removed from the queue for the following reason: pull request manually updated.

The pull request #7741 has been manually updated.

If you want to requeue this pull request, you can post a @mergifyio requeue comment.

[sanderegg]

thanks

[GitHK]

🪄

[odeimaiz]

@Mergifyio queue

[mergify]

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at e5ef303

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud