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

Add Customize Add Page proposal #669

Merged

Conversation

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 25, 2021
@jerolimov jerolimov force-pushed the customize-add-page branch 2 times, most recently from b21f1d5 to 6ea9595 Compare March 18, 2021 18:28
@jerolimov jerolimov changed the title [WIP] Add Customize Add Page proposal Add Customize Add Page proposal Mar 18, 2021
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 18, 2021
@jerolimov
Copy link
Member Author

/uncc imcleod jcantrill
/cc christianvogt spadgett jhadvig
/assign christianvogt spadgett

Copy link
Contributor

@christianvogt christianvogt left a comment

Choose a reason for hiding this comment

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

Overall looks good. I never heard of a requirement to handle groups and PM / UX haven't chimed in yet to any inquiries. I think for the first pass we only support actions unless discussions resume and suggest otherwise.

enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
@jerolimov
Copy link
Member Author

@christianvogt See update above. Hope its fine to keep these questions with the answers in the doc.

Anything else?

Copy link
Contributor

@christianvogt christianvogt left a comment

Choose a reason for hiding this comment

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

The term option and action are interchanged in this doc. Could we be consistent with how we name the add page entries.

enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
@jerolimov
Copy link
Member Author

@christianvogt Pushed a commit with changes based on your recommendation. I hope that I don't add to many new issues. Wdyt?

enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
enhancements/console/customize-add-page.md Outdated Show resolved Hide resolved
@jerolimov
Copy link
Member Author

jerolimov commented Apr 8, 2021

@christianvogt Take all your last typo fixes 1:1 and rebased this changes to one commit.

@christianvogt
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 8, 2021

The admin needs to know the current available actions to hide them. We don't want to provide the defaults in the CR so that we can change these defaults later without migrating the customer data.

To provide the default values close to the editor, the console will provide a code snippets in the sidebar of the YAML editor when editing the `Console` operator config. The snippets should include the IDs of all actions.
Copy link
Member

Choose a reason for hiding this comment

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

We probably need more than a snippet to document these, particularly since a lot of admins use the CLI. Have we given thought about where else we can publish these? Maybe in the console customization topic in the OpenShift docs and the API docs for the property.

Have we considered a simple UI for admins to update them (even if not in 4.8)? Maybe a modal similar to what we have for table column customization.

Copy link
Contributor

Choose a reason for hiding this comment

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

A UI is being considered in future however plans yet in 4.8 or 4.9. This may fall under the umbrella of the same mechanism that will serve customized dashboards.

I agree that we can work with docs to provide guidance on customizing this and other configurations similar to it.

Copy link
Member Author

Choose a reason for hiding this comment

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

@spadgett @christianvogt Should we discuss this in slack and open a ticket in the epics for this? I also expect that we should do this for all 4 (!) customization options we will have in 4.8.

  1. dev catalog categories
  2. available cluster roles to assign on the project page (project access)
  3. disable actions on the add page
  4. disable quick starts

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can move forward with this enhancement and provide docs.

@spadgett
Copy link
Member

spadgett commented Apr 9, 2021

/approve

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: spadgett

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 9, 2021
@openshift-merge-robot openshift-merge-robot merged commit 48b6a7b into openshift:master Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants