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

Operation experience optimization #733

Open
1 of 5 tasks
jinpy666 opened this issue Nov 9, 2020 · 11 comments
Open
1 of 5 tasks

Operation experience optimization #733

jinpy666 opened this issue Nov 9, 2020 · 11 comments
Assignees
Labels

Comments

@jinpy666
Copy link

jinpy666 commented Nov 9, 2020

Please answer these questions before submitting your issue.

  • Why do you submit this issue?
  • Question or discussion
  • Bug
  • Requirements
  • Feature or performance improvement
  • Other

Requirements or improvement

image
It is recommended to merge the two input to input URL instead,you don't have to worry about HTTP/HTTPS or IP/Domain or various query strings.

@membphis
Copy link
Member

membphis commented Nov 9, 2020

It is recommended to merge the two input to input URL instead,you don't have to worry about HTTP/HTTPS or IP/Domain or various query strings.

yes, that is a nice way. ^_^

@juzhiyuan
Copy link
Member

Ok then those 2 fields are related to the API, if we merge them then we should split again before sending the request.

But this change looks good to me!

@juzhiyuan juzhiyuan added the good first issue Good for newcomers label Nov 24, 2020
@juzhiyuan juzhiyuan removed the discuss label Jan 4, 2021
@juzhiyuan
Copy link
Member

@jinpy666 Would you like to take this issue? we could help you if needed.

@mangoGoForward
Copy link
Contributor

I'd like work on this issue.

@juzhiyuan
Copy link
Member

Assign to you :)

@mangoGoForward
Copy link
Contributor

I'm sorry, I have no more time to work on this issue, if others want to try, please assign to yourself.

@mangoGoForward mangoGoForward removed their assignment Jan 17, 2022
@SkyeYoung
Copy link
Member

SkyeYoung commented Jan 18, 2022

It seems that the design of this part has changed a lot.

I don't know, does the part mentioned in the issue now correspond to this look?

image

Does the description of this issue need some updates? And is it still a feature requirement?

@juzhiyuan
Copy link
Member

cc @zaunist

@zaunist
Copy link
Contributor

zaunist commented Jan 18, 2022

@SkyeYoung. You are welcome to submit PRs to contribute. But be careful, after combining the two fields on the form, you still need to split them when sending the request to the backend, because apisix upstream health check need host and http_path fields. ref health check

@Si-ege
Copy link
Contributor

Si-ege commented Feb 21, 2022

I want to try and finish it

@juzhiyuan
Copy link
Member

Hi @Si-ege, if there has no PRs link to this issue, you could start directly :)

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

Successfully merging a pull request may close this issue.

7 participants