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.

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

Changes edit view to json read-only view #112034

Merged
merged 25 commits into from
Sep 21, 2021
Merged

Changes edit view to json read-only view #112034

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
74e910a
Changes edit view to json read-only view
TinaHeiligers Sep 3, 2021
0b25760
Adds copy to clipboard functionality
TinaHeiligers Sep 13, 2021
238f6da
Adds link to saved objects api for not found error message
TinaHeiligers Sep 13, 2021
9dfde70
Update i18n
TinaHeiligers Sep 13, 2021
8e31205
Cleans up todo comments
TinaHeiligers Sep 14, 2021
148a2b9
Merge branch 'master' into so-management/read-only-inspect-view
TinaHeiligers Sep 14, 2021
1110ff3
Updates test for ensuring we can inspect a saved object in a non-defa…
TinaHeiligers Sep 14, 2021
8c92317
Updates unit test snapshot and refactors security-related saved objec…
TinaHeiligers Sep 14, 2021
e2c7143
Merge branch 'master' into so-management/read-only-inspect-view
TinaHeiligers Sep 15, 2021
e091640
Merge branch 'master' into so-management/read-only-inspect-view
TinaHeiligers Sep 16, 2021
b31a0a8
removes getEditUrl from saved object loader searchSavedObjectType, cr…
TinaHeiligers Sep 16, 2021
3c89d0b
Addresses draft comments
TinaHeiligers Sep 16, 2021
3f2b30d
improves functional tests
TinaHeiligers Sep 16, 2021
dac2f76
Fixes som api_integration test
TinaHeiligers Sep 17, 2021
8df5007
Updates unit test snapshots, sets available in relationships flyout a…
TinaHeiligers Sep 17, 2021
9de3eb1
Updates api_integration test making editUrl optional in the schema
TinaHeiligers Sep 17, 2021
121283e
Merge branch 'master' into so-management/read-only-inspect-view
kibanamachine Sep 18, 2021
2c19acc
Merge branch 'master' into so-management/read-only-inspect-view
kibanamachine Sep 19, 2021
3cc2504
Merge branch 'master' into so-management/read-only-inspect-view
TinaHeiligers Sep 20, 2021
7b748d0
Removes serviceRegistry from the top most UI layer
TinaHeiligers Sep 20, 2021
ad1d088
moves inline style to scss file
TinaHeiligers Sep 20, 2021
52f2261
Update snapshots
TinaHeiligers Sep 20, 2021
e703655
Moves inline style to stylesheet
TinaHeiligers Sep 21, 2021
940588d
Merge branch 'master' into so-management/read-only-inspect-view
TinaHeiligers Sep 21, 2021
8c75ad8
Addresses nits
TinaHeiligers Sep 21, 2021
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
16 changes: 8 additions & 8 deletions ...ement/public/management_section/object_view/__snapshots__/saved_object_view.test.tsx.snap
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion ...ment/public/management_section/object_view/components/__snapshots__/inspect.test.tsx.snap
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 0 additions & 3 deletions ...t/public/management_section/objects_table/__snapshots__/saved_objects_table.test.tsx.snap
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion ...d_objects_management/public/management_section/objects_table/components/relationships.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ export class Relationships extends Component<RelationshipsProps, RelationshipsSt
icon: 'inspect',
'data-test-subj': 'relationshipsTableAction-inspect',
onClick: (object: SavedObjectWithMetadata) => goInspectObject(object),
available: (object: SavedObjectWithMetadata) => !!object.meta.editUrl,
available: (object: SavedObjectWithMetadata) => !!(object.type && object.id),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The inspect action is now available for all saved object types (we check that the object provided has a type and id to make sure that routing to the Inspect page will be valid). It used to only be available for saved objects that have an editUrl defined but is not the case anymore.

},
],
},
Expand Down
2 changes: 0 additions & 2 deletions test/api_integration/apis/saved_objects_management/find.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -226,8 +226,6 @@ export default function ({ getService }: FtrProviderContext) {
icon: 'visualizeApp',
title: 'Visualization',
hiddenType: false,
editUrl:
'/management/kibana/objects/savedVisualizations/add810b0-3224-11e8-a572-ffca06da1357',
inAppUrl: {
path: '/app/visualize#/edit/add810b0-3224-11e8-a572-ffca06da1357',
uiCapabilitiesPath: 'visualize.show',
Expand Down