Skip to content

feat(Request pane): Add indicators for body and auth in the request pane tabs#7697

Merged
gatzjames merged 2 commits intoKong:developfrom
gatzjames:feature/ins-4129-show-green-dot-when-setting-up-body-and-auth
Jul 12, 2024
Merged

feat(Request pane): Add indicators for body and auth in the request pane tabs#7697
gatzjames merged 2 commits intoKong:developfrom
gatzjames:feature/ins-4129-show-green-dot-when-setting-up-body-and-auth

Conversation

@gatzjames
Copy link
Contributor

Highlights:

  • Adds indicators [•] to the Body and Auth tabs in the request panes
  • Auth is considered empty only if set to None. Inherit from parent is considered not empty
  • Body is considered empty only if set to No Body

@gatzjames gatzjames self-assigned this Jul 10, 2024
@gatzjames gatzjames requested a review from a team July 10, 2024 13:20
getContentTypeFromHeaders(activeRequest.headers) ||
activeRequest.body.mimeType;
const isBodyEmpty = Boolean(typeof activeRequest.body.mimeType !== 'string' && !activeRequest.body.text);
const hasAuthentication = getAuthObjectOrNull(activeRequest.authentication)?.type !== 'none';
Copy link
Contributor

@jackkav jackkav Jul 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inherit from parent is considered not empty

inherit from parent is the default state, so perhaps this should read isNoneOrInherited rather than hasAuthentication

@gatzjames gatzjames force-pushed the feature/ins-4129-show-green-dot-when-setting-up-body-and-auth branch 3 times, most recently from 2faac18 to 914bd3e Compare July 12, 2024 10:26
@gatzjames gatzjames requested a review from jackkav July 12, 2024 10:44
@jackkav jackkav force-pushed the feature/ins-4129-show-green-dot-when-setting-up-body-and-auth branch from 914bd3e to 7df4461 Compare July 12, 2024 10:44
@gatzjames gatzjames enabled auto-merge (squash) July 12, 2024 10:53
@gatzjames gatzjames merged commit 6e432ad into Kong:develop Jul 12, 2024
@gatzjames gatzjames deleted the feature/ins-4129-show-green-dot-when-setting-up-body-and-auth branch July 12, 2024 10:56
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.

3 participants