-
Notifications
You must be signed in to change notification settings - Fork 37
[Discover] Refactor SetupAccess Screens #1310
Conversation
packages/teleport/src/Discover/Database/SetupAccess/SetupAccess.tsx
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Kubernetes/SetupAccess/SetupAccess.tsx
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Shared/SetupAccess/useUserTraits.ts
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Shared/SetupAccess/useUserTraits.ts
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Shared/SetupAccess/useUserTraits.ts
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Shared/SetupAccess/useUserTraits.ts
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Shared/SetupAccess/useUserTraits.ts
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Shared/SetupAccess/SetupAccessWrapper.tsx
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Server/SetupAccess/SetupAccess.tsx
Outdated
Show resolved
Hide resolved
packages/teleport/src/Discover/Server/SetupAccess/SetupAccess.tsx
Outdated
Show resolved
Hide resolved
c203961
to
adb8723
Compare
packages/build/.eslintrc.js
Outdated
'react/jsx-no-useless-fragment': 2, | ||
|
||
'react-hooks/rules-of-hooks': 'warn', | ||
'react-hooks/exhaustive-deps': 'warn', |
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.
Maybe keep this syntax consistent, so swap these for 1
's
|
||
useEffect(() => { | ||
fetchUserTraits(); | ||
}, []); |
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.
If leaving the dependencies blank is intentional can you add a comment and a //eslint ignore
label here so a future dev knows this was an intentional decision.
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.
authType: 'local', | ||
userName: 'llama', | ||
accessCapabilities: { | ||
suggestedReviewers: ['george_washington@gmail.com', 'alpha'], |
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.
Ahead of his time... 😉
d33e79a
to
c0914fe
Compare
@JanKaczmarkiewicz i moved adding new eslint rules and fixing lint errors in this PR: #1327 |
- create a reusable hook related to setup access and user traits - create a setup access wrapper that dedups repeated logic - rename and added more yaml readers
c0914fe
to
0510791
Compare
@JanKaczmarkiewicz friendly ping |
* Create user trait hook and setup access wrapper - create a reusable hook related to setup access and user traits - create a setup access wrapper that dedups repeated logic - rename and added more yaml readers * Create SetupAccess screen for database * Refactor SetupAccess for kube * Refactor SetupAccess for server * Add database json vals and prevent infinite rendering from dependency array
* Create user trait hook and setup access wrapper - create a reusable hook related to setup access and user traits - create a setup access wrapper that dedups repeated logic - rename and added more yaml readers * Create SetupAccess screen for database * Refactor SetupAccess for kube * Refactor SetupAccess for server * Add database json vals and prevent infinite rendering from dependency array
* Create user trait hook and setup access wrapper - create a reusable hook related to setup access and user traits - create a setup access wrapper that dedups repeated logic - rename and added more yaml readers * Create SetupAccess screen for database * Refactor SetupAccess for kube * Refactor SetupAccess for server * Add database json vals and prevent infinite rendering from dependency array
* Create user trait hook and setup access wrapper - create a reusable hook related to setup access and user traits - create a setup access wrapper that dedups repeated logic - rename and added more yaml readers * Create SetupAccess screen for database * Refactor SetupAccess for kube * Refactor SetupAccess for server * Add database json vals and prevent infinite rendering from dependency array
* Create user trait hook and setup access wrapper - create a reusable hook related to setup access and user traits - create a setup access wrapper that dedups repeated logic - rename and added more yaml readers * Create SetupAccess screen for database * Refactor SetupAccess for kube * Refactor SetupAccess for server * Add database json vals and prevent infinite rendering from dependency array
part of gravitational/teleport#16858
Note: Easier to review by commit, tried to contain specific changes. I think half of the changes are file deletes, and simple yaml file addition and changes.
Description
TODO
Figma - DB setup access screen
https://www.figma.com/file/8DQZl1mYUPLgvKgmA6zPOM/In-Progress-Work?node-id=1299%3A69270
Screenshot
Looks pretty sim to figma, all other states are available in storybook under
Teleport/Discover/*