-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
refactor: isolate computed header calculation on effective requests #2313
refactor: isolate computed header calculation on effective requests #2313
Conversation
Co-authored-by: Andrew Bastin <andrewbastin.k@gmail.com>
Co-authored-by: liyasthomas <liyascthomas@gmail.com> Co-authored-by: Andrew Bastin <andrewbastin.k@gmail.com>
This pull request introduces 3 alerts when merging c41641c into c36e421 - view on LGTM.com new alerts:
|
This pull request introduces 4 alerts when merging 4b88481 into c36e421 - view on LGTM.com new alerts:
|
This pull request introduces 4 alerts when merging 154dff0 into 450af98 - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
- Not gonna put a "Computed header" title, occupies too much space.
- Will iterate over the current UI if any users raise concerns.
This pull request introduces 4 alerts when merging ea82bff into 450af98 - view on LGTM.com new alerts:
|
…oppscotch#2313) Co-authored-by: liyasthomas <liyascthomas@gmail.com>
…2313) Co-authored-by: liyasthomas <liyascthomas@gmail.com>
Description
This PR introduces some refactors and UX improvements to show the headers and parameters that are added indirectly to the request. For example, when you set to Bearer Token Authorization, the Authorization header is added during the execution of the request, the goal of this PR is to refactor those calculations to be independently exposed and to have a UX for showing the user this.
Progress
Checks
Additional Information