Skip to content

feat: add native-rendered Electron app #491

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

Merged
merged 5 commits into from
Aug 29, 2023
Merged

Conversation

lihebi
Copy link
Collaborator

@lihebi lihebi commented Aug 29, 2023

There are two ways to make the electron app:

  • [Approach 1, server-based, preferred]: Host UI at localhost:3000 and API servers at localhost:4000, and use Electron window as a browser. This approach makes it possible to open the app in the browser, so that

    • users can access the browser extensions.
    • lightweight, no need to render another embedded Chromium (the Electron renderer)
    • This approach should also make it easy to re-use the server-app logic.
  • [Approach 2, native-rendered]: Let host only API server on localhost:4000, and render the UI pages & components directly in Electron API. This is the regular way of using Electron.

This PR implements approach 2. It can now render the page without a problem. However, approach 1 should be the way to go. Therefore, this PR is only for reference.

@lihebi lihebi marked this pull request as draft August 29, 2023 01:38
@lihebi lihebi marked this pull request as ready for review August 29, 2023 01:39
@lihebi
Copy link
Collaborator Author

lihebi commented Aug 29, 2023

this PR is only for reference.

Actually, this PR is a general platform to build approach 1 as well. I'm merging this.

@lihebi lihebi changed the title [prototype] add native-rendered Electron app [feat] add native-rendered Electron app Aug 29, 2023
@lihebi lihebi changed the title [feat] add native-rendered Electron app feat: add native-rendered Electron app Aug 29, 2023
@lihebi lihebi merged commit e962717 into codepod-io:main Aug 29, 2023
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.

1 participant