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

Basic web ui features #63

Merged
merged 21 commits into from
Aug 1, 2017
Merged

Basic web ui features #63

merged 21 commits into from
Aug 1, 2017

Conversation

jijiechen
Copy link
Collaborator

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.

@Jaben
Copy link
Member

Jaben commented Jul 10, 2017

Wow! This looks very cool! Can the WebAPI server be run from the service as well as the front-end?

@jijiechen
Copy link
Collaborator Author

jijiechen commented Jul 10, 2017

@Jaben Very good point in your question.
The web UI is booted up by a handler of 'PapercutServiceReadyEvent' event, it seems this event is only fired in the Windows service, but not the desktop app.
I think we can have a discussion on how we can make it work for both desktop mode and background mode.

Jijie added 2 commits July 10, 2017 23:50
…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
@jijiechen
Copy link
Collaborator Author

Now the web ui is launched with both desktop mode and Windows service mode.

@Jaben
Copy link
Member

Jaben commented Aug 1, 2017

@jijiechen This is a wonderful effort. Really excellent. Merging.

@Jaben Jaben merged commit 8af8221 into ChangemakerStudios:develop Aug 1, 2017
@jjchen-tw
Copy link
Contributor

@Jaben Sorry for breaking the CI. I'll take some time on the issues as well.

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

3 participants