HEBorn icon indicating copy to clipboard operation
HEBorn copied to clipboard

[POC] Automation for Electron apps

Open chrisfls opened this issue 6 years ago • 5 comments

This is kinda ready, the only leftover is to fix the API url.

The code is building an app for every architecture of platform, including mas (we'll probably ditch "Mac Apple Store" because steam)

Not sure if it should be merged, I think it's HE2 only, so it's probably not going to be merged now, but kept as a basis for the future (I might be wrong).

Note: Most of the names are HE2 related (the .exe name is Hacker Experience 2.exe).


This change is Reviewable

chrisfls avatar Mar 16 '18 22:03 chrisfls

Didn't expected this so soon, but it seems fine!


Reviewed 5 of 5 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

PedroHLC avatar Mar 17 '18 17:03 PedroHLC

It's not ready because I don't know what values we should use for the flags.

chrisfls avatar Mar 18 '18 04:03 chrisfls

Changes:

  • Added electron dependency to package.json, I've assumed it wasn't needed and removed it before, but it's really needed;
  • Added support for running the app with development server (read electron.js and static/main.js), it has more to do with invalid certificate problems and spawning the devtools;
  • Added electron.js with build instructions, it replaces the previous electron-packager in the Makefile, the main reason for that is to moves files around to fix the problem bellow;
  • Fixed static file loading, now images load just fine like on production (can't say the same for the wallpaper);

chrisfls avatar Mar 21 '18 05:03 chrisfls

And now bsd compatibility is biting us again :cry:

chrisfls avatar Mar 21 '18 05:03 chrisfls

Note: this is really almost ready to merge, additional comments on our private discord dev channel.

chrisfls avatar Mar 21 '18 06:03 chrisfls