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

Add configurations #17

Open
empijei opened this issue Oct 2, 2017 · 2 comments
Open

Add configurations #17

empijei opened this issue Oct 2, 2017 · 2 comments

Comments

@empijei
Copy link
Owner

empijei commented Oct 2, 2017

Move all possible configurable parameters to config package

@empijei empijei added this to the Initial Stage milestone Oct 2, 2017
@empijei empijei self-assigned this Oct 2, 2017
@mmachado95
Copy link

Is anyone working on this?
Can I do it?

@empijei
Copy link
Owner Author

empijei commented Oct 3, 2017

No one that I know of is working on this.

The main point is that wapty will load a configuration on start.

The configuration is going to contain several recent projects.

A project currently is just a list of SaveLoadStringer and should be changed to a struct containing the current list of saveloadstringer and to also contain 2 more fields: a list of strings or ints(representing ports) for the proxy to listen on and a single port for the UI to listen on.

The first list of ports is going to be used here (in the future wapty will load more than just one proxy) and the UI port is going to be used here

The config package should export functions to get the the ports configured in the currently loaded project, which you can just write a stub for, that will, for now, just return the default values.

To achieve so please make the loaded project return default values if the ports are not set.

Please let me know if it is clear, if so I'm gonna flag this as "inprogress"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants