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

Added Table to global settings #308

Merged
merged 2 commits into from
May 24, 2023

Conversation

andycandy-de
Copy link
Contributor

Hello, in some cases it's useful to configure the Table setting of the wg config. This is possible with this change. This change allows you to define a specific table for the wg routing. Or you can disable the auto routing with Table = off. This can be helpful if you have some special routing rules in your PostUp and PostDown scripts.

Would be great if you can merge this request and release a new docker image. Thank you for this great project.

Added Table to UI global settings view
Added Table to wg conf template
Added Table to settings model
Added Table with default value auto to config util Updated README

Added Table to UI global settings view
Added Table to wg conf template
Added Table to settings model
Added Table with default value `auto` to config util
Updated README
@ngoduykhanh
Copy link
Owner

Sorry for the late review. Can you update your PR and resolve the conflicts?

# Conflicts:
#	README.md
#	model/setting.go
#	store/jsondb/jsondb.go
#	templates/global_settings.html
#	util/config.go
@andycandy-de
Copy link
Contributor Author

Done

@stale
Copy link

stale bot commented May 21, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 21, 2023
@andycandy-de
Copy link
Contributor Author

This PR is still ready to merge

@stale stale bot removed the stale label May 23, 2023
@ngoduykhanh ngoduykhanh merged commit e9357d8 into ngoduykhanh:master May 24, 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.

2 participants