FDK Validation Fix Code Thread

BrianDev · October 4, 2024, 4:29am

Good day, Freshdesk Developers Team.

I like to raise concern regarding acceptable code bases in your platform.

We have this lint error coming from “fdk validation --fix” that we see this error on the parsing error:

And what I noticed on the codebase, we used this kind of scenario that can append HTML tags and this was developed a long time ago.

Asking for this one if do we have alternatives to fix this? Reason is because it is essential to append this tag because of usability with our UI.

FDK used:
v9.1.3

Custom App: Freshdesk Custom App

Let us know if you need to align with me since we will fix lint errors for our tech initiative to avoid blockers on our future releases.

Thank you.

Regards,
Bryan

BrianDev · October 4, 2024, 4:46am

I tried this only like this:

Screenshot 2024-10-04 at 12.43.24 PM

Still gives this error. Not sure why this treats this as lint error as this is normal on React.JS per link here: Common components (e.g. <div>) – React

BrianDev · October 4, 2024, 4:57am

I did also tried to remove the line of code on line 352 that screenshotted:

It gives this error that is confusing to me because it is treated like React JS HTML tags as parsing error since this to render the class name of the UI Screen Function named < AccessRights >.

BrianDev · October 4, 2024, 5:36am

Not sure if this can be scope but I see this old Freshworks Git from Freshdesk Folder Section that uses html React Tags:

github.com

freshworks/marketplace-sample-apps/blob/7df31f9fe2d875109b259cacee595bc365705777/Demo-Apps/react_todo_app/app/app.js#L16


      
                  ReactDOM.render(<div>
                    <Todolist />
                  </div>
                    , document.getElementById('mydiv'))
                });
            });
          
          /**
           * React Component to list todo's 
           */
          class Todolist extends React.Component {
            constructor(props, context) {
              super(props, context);
              this.state = {
                todos: []
              };
              this.handleChecked = this.handleChecked.bind(this);
              this.getList = this.getList.bind(this);
              this.getList()
            }

Raviraj · October 11, 2024, 9:27am

Hi @BrianDev,

Are you using the react component tag in the app.js file or the modal js files?

Could you share a simplified version of the code that you’re trying with us to test it?

BrianDev · October 16, 2024, 1:25am

Hi, Sir @Raviraj!

I sent you an email regarding this one. Let me know if you have concerns.

Thank you

system · January 14, 2025, 1:26am

This topic was automatically closed 90 days after the last reply. New replies are no longer allowed.