Skip to content
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

UI with discover scopes #125

Merged
merged 6 commits into from
Mar 12, 2023
Merged

UI with discover scopes #125

merged 6 commits into from
Mar 12, 2023

Conversation

akpsgit
Copy link
Contributor

@akpsgit akpsgit commented Mar 10, 2023

All UI code introduced in this commit. This also contains scopes discovery and adjustment of existing tables to use the new oData I/S

@akpsgit akpsgit requested a review from a team as a code owner March 10, 2023 15:15
ghost
ghost previously requested changes Mar 10, 2023
backend/pkg/database/database.go Outdated Show resolved Hide resolved
backend/pkg/database/gorm/common.go Outdated Show resolved Hide resolved
backend/pkg/database/gorm/odata.go Outdated Show resolved Hide resolved
backend/pkg/database/gorm/odata.go Show resolved Hide resolved
api/openapi.yaml Outdated Show resolved Hide resolved
backend/pkg/database/gorm/resttodb_test.go Outdated Show resolved Hide resolved
backend/pkg/rest/scan_result_controller.go Outdated Show resolved Hide resolved
backend/pkg/rest/scan_result_controller.go Outdated Show resolved Hide resolved
backend/pkg/rest/server.go Show resolved Hide resolved
backend/pkg/rest/target_controller.go Outdated Show resolved Hide resolved
@FrimIdan FrimIdan requested a review from a user March 12, 2023 10:46
@akpsgit akpsgit requested review from a user and removed request for a user March 12, 2023 11:08
@akpsgit akpsgit dismissed ghost ’s stale review March 12, 2023 11:16

Great review, everything was addressed. Just need to f-ing push it

@akpsgit akpsgit merged commit ba83761 into main Mar 12, 2023
@FrimIdan FrimIdan deleted the discover_scopes branch April 20, 2023 08:17
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.

None yet

2 participants