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

Photino Development Server #4

Open
wants to merge 23 commits into
base: debug
Choose a base branch
from
Open

Conversation

Denny09310
Copy link

Adds the architecture to support a development server for frontends frameworks. The sample is built with react and vite, but it should work with any framework as long as the command is set to the right script (default: npm run dev)

Copy link
Contributor

@MikeYeager MikeYeager left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make 2 separate pull requests. One for changes to Photino.NET.Server. Another for the new sample project. The sample project should be added to the Photino.Samples repo. Thanks for your efforts!

@Denny09310
Copy link
Author

Please make 2 separate pull requests. One for changes to Photino.NET.Server. Another for the new sample project. The sample project should be added to the Photino.Samples repo. Thanks for your efforts!

The sample project is not the one in this solution. I can remove it from the pull request as this is to test only the dev server. The complete sample project has already a pull request for the right repository.

@MikeYeager
Copy link
Contributor

@Denny09310 The reference to Photino.NET should be a NuGet reference, not a Project reference.

@Denny09310
Copy link
Author

Denny09310 commented Mar 22, 2024

@MikeYeager There is a problem for now in swapping the project reference with the nuget package. As I've mentioned in the discussion update the namespace has been corrected from PhotinoNET to Photino.NET, this causes a misalignment. This pull request can wait until the new release, or if there is an alternative way I'll be more than happy

Updated Photino.NET NuGet package version to latest.
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

Successfully merging this pull request may close these issues.

2 participants