-
Notifications
You must be signed in to change notification settings - Fork 278
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
Basic web ui features #63
Conversation
…ges for Papercut.
…o the FactBase class
…erence inconsistency problems.
…ted functionalities
Wow! This looks very cool! Can the WebAPI server be run from the service as well as the front-end? |
@Jaben Very good point in your question. |
…I; Fixed issues in AssemblyScanner when a plugin has a customized reference other than pre-referenced assemblies; Renamed Papercut.WebUI projects to Papercut.Module.WebUI projects
…put startup messages instead of Console.WriteLine
Now the web ui is launched with both desktop mode and Windows service mode. |
@jijiechen This is a wonderful effort. Really excellent. Merging. |
@Jaben Sorry for breaking the CI. I'll take some time on the issues as well. |
In these change sets, we were trying to add a web UI for Papercut.
We added a Papercut.WebUI plugin project and the server will be booted by the "PapercutServiceReadyEvent" event. To ensure our implementation works as expected, we also added a test project, the Papercut.WebUI.Test project, which was powered by xunit.
The frontend material came from the MailHog project, it was powered by AngularJS 1.3 version. Despite the framework is now a bit out dated, we just reused it to make our work done sooner.
Additionally, we changed some small codes to add testability for the solution. Thanks to the DI framework and event based architecture, this work was done easily.
Please review and give your feedback, and merge this into trunk at your convenience.
Finally, many thanks to the great idea of Papercut, and the possibilities it gives to people.
@amymengfan @Stack-Coding Please subscribe to this thread as well.