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

Feature/JA-114 Blazor components configuration #106

Merged
merged 8 commits into from
Jun 26, 2024

Conversation

Zjyslav
Copy link
Collaborator

@Zjyslav Zjyslav commented Jun 5, 2024

They are actually Razor components, but it's not important.

I configured the minimum we will need to use components with all available render modes (server or WASM, prerendered or not).

We should put all components in TutorLizard.Blazor.Components, unless we really need them to be run in Server mode and have access to other assemblies (most likely won't happen), then we'll put them in TutorLizard.Web.Components.

To use a Razor component in a .cshtml view, all we need is this snippet:

<component type="typeof(OurComponent)" render-mode="WebAssemblyPrerendered"/>

@Zjyslav Zjyslav marked this pull request as draft June 9, 2024 17:02
@Zjyslav
Copy link
Collaborator Author

Zjyslav commented Jun 10, 2024

I added a new project: TutorLizard.Shared where I moved all our DTOs and Enums (the ones that were in TutorLizard.BusinessLogic.Enums).
Since this project doesn't have a reference to the BL, we cannot use our DB models, so I removed some unused DTOs that referenced them and modified the ones that were used to ask only for specific values.
Then, I renamed some methods in services and the requests/response DTOs, so now they have consistent names (not necessarily better, but consistent).

I also wanted to do a small refactoring of DTOs used for logging in and user registration, but I worry it might cause too much conflicts for OAUTH that's not merged yet. I'll get to it, eventually.

@Zjyslav Zjyslav marked this pull request as ready for review June 10, 2024 18:08
@skrawus skrawus merged commit ec1435d into develop Jun 26, 2024
1 check passed
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.

None yet

2 participants