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

Setting subs-group name from the optional remarks query parameter in the pasted subs url #5264

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

OnceUponATimeInAmerica
Copy link
Contributor

@OnceUponATimeInAmerica OnceUponATimeInAmerica commented Jun 22, 2024

This PR adds support for setting the subs-group remarks property (group's name) from the optional remarks query parameter in the pasted subs url.

If no query parameter is supplied in the subs url then, subs group's remarks property is set to the default/old value of import_sub (to make it fully compatible with the old behavior).

For example, if the url http://example.com/subs/example-id?remarks=BestVpn is pasted in v2rayN, then will show as:

2024_06_22_13_43_53_v2rayN_V6 45_2024_06_22_Not_run_as_Admin

…nal) "remark" query parameter in the pasted subs url
@OnceUponATimeInAmerica OnceUponATimeInAmerica changed the title Setting subs-group name from the optional "remarks" query parameter in the pasted url Setting subs-group name from the optional "remarks" query parameter in the pasted subs url Jun 22, 2024
@OnceUponATimeInAmerica OnceUponATimeInAmerica changed the title Setting subs-group name from the optional "remarks" query parameter in the pasted subs url Setting subs-group name from the optional remarks query parameter in the pasted subs url Jun 22, 2024
@OnceUponATimeInAmerica OnceUponATimeInAmerica changed the title Setting subs-group name from the optional remarks query parameter in the pasted subs url Setting subs-group name from the optional remarks query parameter in the pasted subs url Jun 22, 2024
@2dust 2dust merged commit a28c631 into 2dust:master Jun 24, 2024
1 check passed
2dust added a commit that referenced this pull request Jun 24, 2024
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

2 participants