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

Fiori Element Add column to table missing crucial information #110

Closed
NSXBLUE opened this issue Feb 22, 2024 · 12 comments
Closed

Fiori Element Add column to table missing crucial information #110

NSXBLUE opened this issue Feb 22, 2024 · 12 comments
Assignees
Labels
contribution Valuable Contribution SAP Fiori elements type/content-gaps Something essential is missing in the documentation.

Comments

@NSXBLUE
Copy link

NSXBLUE commented Feb 22, 2024

Issue description

The documentation of extending Fiori Element responsive table is missing the name of the fragment files you are supposed to create. It mentions you need two extensions, then it provides code for the two extensions, but it only specifies the file name for the second code snippet. It took me hours to find out what I was doing wrong.

https://sapui5.hana.ondemand.com/#/topic/28e95702b5854b938ac51c4bc2d078ab.html

The first code snippet needs to mention create a file: webapp/ext/fragments/ListReportResponsiveTableCells.fragment.xml

The equivalent doc for grit table extension is also missing this crucial piece of information.

Also, the documentation fails to indicate how to deploy this FE extension and run it in Fiori Launchpad. How do you ensure your new columns will be available when users click on the original app's tile?
I'm still stuck on that, trying to find other online sources to explain the process.

Thanks

Feedback Type (Optional)

examples or samples

Page Title on SAP Help Portal (prefilled)

Example: Adding Columns to a Responsive Table in the List Report

Page URL on SAP Help Portal (prefilled)

https://sapui5.hana.ondemand.com/#/topic/28e95702b5854b938ac51c4bc2d078ab.html

@OlMue
Copy link
Contributor

OlMue commented Feb 23, 2024

Hi @NSXBLUE, thanks for your contribution! We'll look into it and get back to you.

@mishuagrawal
Copy link
Contributor

Hello @NSXBLUE, we are working on your concern and will share an update here soon. Kindly bear with us while we try to resolve this. Thank you!

@anjan-poonacha
Copy link

@mishuagrawal I could add the following code in my test repository. As a result a custom column is visible.

<!-- Generate the custom column fragment -->
<!-- fragments/ResponsiveTableColumns.fragment.xml -->
<core:FragmentDefinition xmlns:core="sap.ui.core" xmlns="sap.m">
    <!--Column 1 Header-->
    <Column id="textColumn">
        <Text text="Extension Column"/>
        <customData>
            <core:CustomData key="p13nData" value="\{'columnKey': 'CustomColumn1', 'leadingProperty': 'Depth', 'columnIndex': '101'}"/>
        </customData>
    </Column>
</core:FragmentDefinition>
<!-- Generate custom cell fragment -->
<!-- fragments/ResponsiveTableCells.fragment.xml -->

<core:FragmentDefinition xmlns:core="sap.ui.core" 
    xmlns="sap.m">
	<!--Column 1 Cell-->
	<Text text="Breakout Column"></Text>
</core:FragmentDefinition>
// Add extension settings to the manifest
// manifest.json

"sap.ui.viewExtensions": {
	"sap.suite.ui.generic.template.ListReport.view.ListReport": {
		"ResponsiveTableColumnsExtension|STTA_C_MP_Product": {
			"className": "sap.ui.core.Fragment",
			"fragmentName": "stta.prod.man.typescript.ext.fragments.ResponsiveTableColumns",
			"type": "XML"
		},
		"ResponsiveTableCellsExtension|STTA_C_MP_Product": {
			"className": "sap.ui.core.Fragment",
			"fragmentName": "stta.prod.man.typescript.ext.fragments.ResponsiveTableCells",
			"type": "XML"
		}
	}
}

image

@mishuagrawal
Copy link
Contributor

Hello @NSXBLUE,

Thank you for your patience. As you can see, our dev has reviewed your concern and they have confirmed that they were able to add the mentioned code in the test repository and were also able to create a custom column. If you are still facing issues, please let us know.

Also, regarding the deployment of FE extension to SAP Fiori launchpad, kindly check the Deploying an Application (https://help.sap.com/docs/SAP_FIORI_tools/17d50220bcd848aa854c9c182d65b699/1b7a3be8d99c45aead90528ef472af37.html) documentation. This document is for SAP Fiori Tools.

Please let us know if this resolves your query.

Thanks and regards,
Mishu Agrawal

@mishuagrawal
Copy link
Contributor

Hello @NSXBLUE,

We are waiting to hear from you. Kindly write back to us and let us know how we can help you further in this regard.

Best,
Mishu Agrawal

@NSXBLUE
Copy link
Author

NSXBLUE commented Mar 21, 2024

I guess I wasn't clear about the issue, sorry. What I'm saying is that there are two pieces of code for two fragments. One of them (the second one) clearly specifies the file name and location it needs to be saved in: /webapp/ext/fragments/ListReportResponsiveTableColumns.fragment.xml. But the first one does not.

Please see screenshot below. What is the name and location of the first fragment?

image

@NSXBLUE
Copy link
Author

NSXBLUE commented Mar 21, 2024

If you're specifying clearly the name and location of the second fragment, then the first one should also be specified.

Also regarding deployment, the link you provided was for general deployment of apps. What I think should be mentioned here is how to make sure these extensions (fragments) are being deployed, ie so when the original standard tile is executed by the user, our extensions are being executed,.

@mishuagrawal
Copy link
Contributor

Hello @NSXBLUE,

Thank you for writing back to us. This helps. We are now working on your concerns and will get back to you soon.

Thanks and regards,
Mishu Agrawal

@cfg74
Copy link

cfg74 commented May 2, 2024

Hi @NSXBLUE I would like to address your deployment question. When extensions are made to the app in your development environment, then they are part of the app's code, and will be deployed when you redeploy the now modified app. The documentation you are following assumes that the app is your own custom developed application where you own the lifecycle.

In case you want to extend an application which was delivered to you (like a SAP provided app), you have to create an adaptation project which is a different process with also partly different tooling. Please refer to https://sapui5.hana.ondemand.com/#/topic/52fc48b479314d0688be24f699778c47

@mishuagrawal
Copy link
Contributor

Hello @NSXBLUE,

Thank you for your patience. I apologies that it took us some time to resolve your concern.

We have now enhanced the document. We have mentioned the file name and location for both the code fragments now. Regarding your deployment query, my colleagues @cfg74 has responded to it. I hope this resolves your concern.

If yes, I kindly request you to close this issue. If you have any concerns, you can reach out to us.

Thanks and regards,
Mishu Agrawal

@NSXBLUE
Copy link
Author

NSXBLUE commented May 2, 2024

Thanks for your help.
Closing the issue.

@NSXBLUE NSXBLUE closed this as completed May 2, 2024
@OlMue OlMue added contribution Valuable Contribution type/content-gaps Something essential is missing in the documentation. labels Jun 5, 2024
Copy link

sap-doc-bot bot commented Jun 5, 2024

Thank you for your valuable feedback contribution, @NSXBLUE! So that we can recognize your contribution in the SAP Community, please check your SAP Community user ID (this is a number) in your personal settings page and share it with us in a reply to this comment. Make sure you just include the number in the reply.

Your user ID is displayed as follows:

Change display name for User ID N

where N is your user ID. For example, 53 is the user ID of the user 'qmacro'.

Please note that we are currently refactoring our profile and badge system on the SAP Community, and will start assigning badges again when that's complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution Valuable Contribution SAP Fiori elements type/content-gaps Something essential is missing in the documentation.
Projects
None yet
Development

No branches or pull requests

6 participants