Skip to content

feat: condition Audit log on licensing #3685

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.

Sign up for GitHub

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

Merged
merged 21 commits into from
Aug 25, 2022
Merged

feat: condition Audit log on licensing #3685

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
aced91f
Update XService
presleyp Aug 24, 2022
49f5652
Add simple wrapper
presleyp Aug 24, 2022
a641881
Add selector
presleyp Aug 24, 2022
4e79b2d
Condition page
presleyp Aug 24, 2022
6dfe4df
Condition link
presleyp Aug 24, 2022
17d8d1b
Format and lint
presleyp Aug 24, 2022
8fdc834
Integration test
presleyp Aug 24, 2022
3627ef3
Add username to api call
presleyp Aug 24, 2022
8b88265
Format
presleyp Aug 24, 2022
4fea785
Format
presleyp Aug 24, 2022
733954b
Fix link name
presleyp Aug 25, 2022
fc237a0
Upgrade xstate/react to fix crashing tests
presleyp Aug 25, 2022
cb20ccd
Fix tests
presleyp Aug 25, 2022
75f7f6e
Format
presleyp Aug 25, 2022
2e94269
Abstract strings
presleyp Aug 25, 2022
3c54e1e
Debug test
presleyp Aug 25, 2022
4d40c30
Increase timeout
presleyp Aug 25, 2022
a26786e
Add comments and try shorter timeout
presleyp Aug 25, 2022
a7d836b
Use PropsWithChildren
presleyp Aug 25, 2022
4f975b8
Undo PropsWithChildren, try lower timeout
presleyp Aug 25, 2022
9505faa
Format, lower timeout
presleyp Aug 25, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Undo PropsWithChildren, try lower timeout
  • Loading branch information
@presleyp
presleyp committed Aug 25, 2022
commit 4f975b86faf43340d4edfff3d9a42223c2a33220
6 changes: 3 additions & 3 deletions site/src/components/Navbar/Navbar.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ describe("Navbar", () => {
const link = screen.getByText(Language.audit)
expect(link).toBeDefined()
},
{ timeout: 5000 },
{ timeout: 2500 },
)
})

Expand All @@ -40,7 +40,7 @@ describe("Navbar", () => {
const link = screen.queryByText(Language.audit)
expect(link).toBe(null)
},
{ timeout: 5000 },
{ timeout: 2500 },
)
})

Expand All @@ -63,7 +63,7 @@ describe("Navbar", () => {
const link = screen.queryByText(Language.audit)
expect(link).toBe(null)
},
{ timeout: 5000 },
{ timeout: 2500 },
)
})
})
6 changes: 3 additions & 3 deletions site/src/components/RequirePermission/RequirePermission.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import { FC, PropsWithChildren } from "react"
import { FC } from "react"
import { Navigate } from "react-router"

export type RequirePermissionProps = PropsWithChildren<{
export interface RequirePermissionProps {
children: JSX.Element
Copy link
Contributor

Choose a reason for hiding this comment

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

Would PropsWithChildren help here? reference

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried using it here and got a TS error, not sure why!

Copy link
Contributor

Choose a reason for hiding this comment

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

This works in the playground: https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAJQKYEMDGMA0cDecAKUEYAzgOrAwAWAwlcADYAmUSAdnAL5wBmRIcAOSt0MQQFgAUFKQAPSLDgwAnmCSIkARwCuwVviRQQwEiWAQ2hYiTgBeAkVIVqdRi3YAeHFLhwTAMVQYbVYANRNgACMGJAAuOEiICBiUNilOAD4pKTQLEngAORQAN2AAcxQYdXsACjBHEnicGAh4-KhgNjLOAEo7DLgPFDgqVh5bHHrrADoWzIZOgGsPAHoULOlJFYAqbZ9tuDIoFFI4Im0qm2pKuBRWW+KURhRo9UiUEiQmOAtEACEAII0H5QOALNDsMxdab7FYyeTQeC5Nj5DQ6PRIAxGExmX61PBoejMVhsbABIIhJDhMyvLjxZDo-SGYymcyWBp9WwDbySXzAHhwGoAQnJlUp1KiMT6PN8vlYwSgHA8RVKFSqSggtgARCsAO7QRYkMDoJAkLVwFYbXzcJAMT64Hyy+UhDiEtwkx2cdJAA

Up to you though: just a suggestion

isFeatureVisible: boolean
}>
}

/**
* Wraps routes that are available based on RBAC or licensing.
Expand Down