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

@RenderBody not working on Linux #23

Open
mmkathurima opened this issue Jan 15, 2025 · 10 comments
Open

@RenderBody not working on Linux #23

mmkathurima opened this issue Jan 15, 2025 · 10 comments

Comments

@mmkathurima
Copy link

This first picture of a small demo application is running on a Windows server
image
This second picture is the same application but on Linux server
image
As can be seen, the content of the @RenderBody, in this case the contents of Index.vbxml.vb is not rendered on Linux.
@VBAndCs kindly assist on this.

@VBAndCs
Copy link
Owner

VBAndCs commented Jan 15, 2025 via email

@mmkathurima
Copy link
Author

@VBAndCs I have looked into the issue and I believe it lies in the ZML module, specifically the ZMLExtentions.vb file at the line:

Public Const Ln = vbCrLf

I suggest changing it to the below, for the sake of compatibility across operating systems:

Public ReadOnly Ln As String = Environment.NewLine

I have not raised a pull request as I do not know how to.

@InteXX
Copy link
Contributor

InteXX commented Jan 19, 2025

@mmkathurima

vbCrLf

That's a good find.

You can raise a PR by editing the file at this repo, which forks the repo to your account. You'll raise the PR from there when you save the changes.

@VBAndCs
Copy link
Owner

VBAndCs commented Jan 19, 2025 via email

@InteXX
Copy link
Contributor

InteXX commented Jan 19, 2025

@VBAndCs

I still has issues to log in to this account

Hm, I thought you fixed that.

@mmkathurima
Copy link
Author

Well noted @InteXX. Kindly feel free to create the PR on my behalf, though I believe @VBAndCs would need his account to approve the PR.

@InteXX
Copy link
Contributor

InteXX commented Jan 20, 2025

@mmkathurima

feel free to create the PR on my behalf

Thank you, and I'd love to, but I'd much rather see you learn how to do a PR ;-)

@VBAndCs
Copy link
Owner

VBAndCs commented Jan 20, 2025 via email

@mmkathurima
Copy link
Author

I have updated dependencies and looks ok so far:

Image

@InteXX
Copy link
Contributor

InteXX commented Jan 20, 2025

@VBAndCs

If you alteady updated your copy of Vazor

I haven't gone past net8.0, as the package for VB.NET design-time EF Core is itself still at net8.0.

The sample project I worked up is here.

What problems did you run into when moving to net9.0?

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

No branches or pull requests

3 participants