[React] Numeric field of an entity is sent as a string in the JSON via REST API

[qmonmert]

Fix #23763

---

Please make sure the below checklist is followed for Pull Requests.

• All continuous integration tests are green
• Tests are added where necessary
• The JDL part is updated if necessary
• jhipster-online is updated if necessary
• Documentation is added/updated where necessary
• Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

[hide212131]

@qmonmert Thank you for addressing this issue. 🙂
I'll comment on the points I've noticed.

[hide212131]

Actually, I'm currently working on a generator for integration tests using Testing Library and MSW to increase test coverage for React entities.
generator-jhipster-react-enhance

It's still under construction, but it generates code like the following:
Generated Code Example

I discovered issues #23763 and #23769 during the development process.
If you'd like to give it a try, I'd be delighted.

[hide212131]

@qmonmert Thank you for addressing the changes. I've checked them and everything looks good. 😃
...However, apologies for the piecemeal feedback 😭 . Upon further investigation, I found another minor issue.

This is the section where related entity IDs are sent.

https://github.com/jhipster/generator-jhipster/blob/72de10a384c263a7af972742669e043de6021282/generators/react/templates/src/main/webapp/app/shared/util/entity-utils.ts.ejs#L41C116-L41C118

The final { id } is a string in the form, but the actual entity ID should be a number.
I think it would be better to use map((id: any) => ({ id: Number(id) }));.

[DanielFran]

Adding a bug bounty, seems lot of work

[qmonmert]

@DanielFran thanks

[qmonmert]

@mshima always the same build fails :/

[mshima]

The failure is related to jdl not loading the databaseType into entity.

It’s considering the postgres (gateway) primary key default (Long instead of UUID).

[mshima]

We should add a warning for microservice entities to use custom id.

[qmonmert]

@mshima thanks for the help 🙂

[qmonmert]

@DanielFran https://opencollective.com/generator-jhipster/expenses/167530 🙂

[DanielFran]

@qmonmert approved