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: Add Blazor implementation of Sextant #200

Merged
merged 35 commits into from
Jan 22, 2020

Conversation

limefrogyank
Copy link
Contributor

What kind of change does this PR introduce?
Adding a Blazor implementation of Sextant.

What is the current behavior?
There is no Blazor implementation yet.

What is the new behavior?
Sextant works on Blazor :)

What might this PR break?
Lots. My branch for Blazor work had some of the UWP stuff that may have been changed. I may have inadvertently committed changes to some core files and I'll need help with understanding git commands to edit those out. (I have no idea how to reverse things without erasing all changes.)

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:
I added README.md at root of the Sextant.Blazor project that explains all. I also added two sample apps (server-side and client-side). The server-side version includes authorization mixed in just to show it works.

@limefrogyank
Copy link
Contributor Author

Yes, all of those conflicting files don't really have any changes... or at least they shouldn't.

@RLittlesII RLittlesII changed the title Blazor implementation of Sextant feature: Add Blazor implementation of Sextant Jan 22, 2020
@RLittlesII RLittlesII changed the base branch from master to feature/blazor January 22, 2020 03:27
@RLittlesII RLittlesII merged commit 00ed44e into reactiveui:feature/blazor Jan 22, 2020
@lock lock bot locked and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants