-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
feat(Global Environments): Introduce a new top level file type called global environments. #7511
Conversation
a20fd61
to
750f9c0
Compare
can you include some screen shots in the description please. I think the ux is a bit contentious. I would also like to see a clear the environment model addressing the current inconsistencies in order to code review this approach. |
7058467
to
0f08cd2
Compare
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.
Went through some parts like data model, rendering and the UI part is not checked yet.
}; | ||
|
||
export const scopeToBgColorMap: Record<string, string> = { | ||
design: 'bg-[--color-info]', | ||
collection: 'bg-[--color-surprise]', | ||
'mock-server': 'bg-[--color-warning]', | ||
unsynced: 'bg-[--hl-md]', | ||
environment: 'bg-[--color-font]', |
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.
nit: Should it be --color-font
or color-bg
?
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.
Yep it's correct we want the color-font for bg here. Agree that it's counter intuitive but there are cases were we need the font color as bg 🫣
f9653fd
to
b98088b
Compare
9113f83
to
40b0bea
Compare
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.
Tend to move forward but there are many details in the component so it may deserve more detailed check.
40b0bea
to
7ef6780
Compare
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 we've discuss several topics like relationship syncing, ux consistency and overall it looks to me. We will also apply it to the scripting feature, have more tests on it and move to the importing/exporting implementation.
… global environments. (Kong#7511) * add new workspace type 'environment' * new route for environments and set/get for them * update environment picker UI * update workspace environment modal ui to match env view * global environment request rendering * type checks * ui overflow fix * update e2e * auto-lint issue * type-checks * Update aria label * create env from project view * changes * save * environment picker component * update e2e
Highlights:
environment
Next steps:
globals
API in pre-request/after-response scriptsNew Environment picker UI:
Global Environments in the Project view:
Updates to Models
environment