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

No hints are displayed when merging requests #468

Open
DarKsandr opened this issue Jul 26, 2024 · 7 comments
Open

No hints are displayed when merging requests #468

DarKsandr opened this issue Jul 26, 2024 · 7 comments

Comments

@DarKsandr
Copy link

I have two requests that I merged into one. When applying the received request, there are no hints.
Снимок экрана от 2024-07-26 11-33-47
Снимок экрана от 2024-07-26 11-34-42
Снимок экрана от 2024-07-26 11-39-15
Снимок экрана от 2024-07-26 11-39-49
Снимок экрана от 2024-07-26 11-40-05

@DarKsandr
Copy link
Author

I thought that double-specifying the request would work, but only the first one that exists is displayed.
Снимок экрана от 2024-07-26 21-03-11
Снимок экрана от 2024-07-26 21-04-33

@DarKsandr
Copy link
Author

@romalytvynenko is there a solution to the problem?

@romalytvynenko
Copy link
Member

@DarKsandr it should be fixed with #237 , but this is not that fast.

@DarKsandr
Copy link
Author

DarKsandr commented Jul 31, 2024

@romalytvynenko got it, thanks. but isn't the second example a bug? shouldn't it combine several requests? i thought it would work the same way as in this example:
Снимок экрана от 2024-07-31 15-51-08
Снимок экрана от 2024-07-31 15-49-25

@romalytvynenko
Copy link
Member

Personally to me this is hard to consider it as a bug, as I never saw such usage of requests. I agree this may be improved, but rather than an improvement, not a bug, IMO

@DarKsandr
Copy link
Author

DarKsandr commented Jul 31, 2024

@romalytvynenko If such an improvement were to appear, I would be glad. But IMHO it is still a bug, because the parser does not take into account all method requests, but only the first one.

@romalytvynenko
Copy link
Member

@DarKsandr feel free to open a PR :-)

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

No branches or pull requests

2 participants