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

Make create/add button consistent on all SPAs #7473

Open
GaneshSPatil opened this issue Dec 19, 2019 · 18 comments
Open

Make create/add button consistent on all SPAs #7473

GaneshSPatil opened this issue Dec 19, 2019 · 18 comments
Assignees
Labels

Comments

@GaneshSPatil
Copy link
Contributor

Issue Type
  • Feature enhancement
Summary

GoCD application has a separate page to manage the different GoCD entities. Each page has either an Add or Create button to create a new entity.
The styles of these add entity buttons are different on each page. Example:

On Pipelines Page:

Screen Shot 2019-12-19 at 11 34 41 AM

On Environments Page:

Screen Shot 2019-12-19 at 11 34 50 AM

On Templates Page:

Screen Shot 2019-12-19 at 11 34 59 AM

On Elastic Agent Configurations Page:

Screen Shot 2019-12-19 at 11 35 08 AM

Expected Results

All GoCD pages should have a consistent Add entity button.

Actual Results

Different GoCD pages implements the Add entity button differently.

Possible Fix

We can choose one of the button styles from the above-mentioned 4 button styles.

@adityasood
Copy link
Contributor

@GaneshSPatil I think the idea specifically for the new Pipelines SPA is that there are two buttons
Create new Pipeline Group and Add New Pipeline. Given the placement of these, @streisguth suggested we keep the primary button in bold and hence Add New Pipeline was made bold.

I agree we should standardize on what the text should read and if we need the plus icon.

Maybe something like Create <entity-name>. @streisguth could you help with some thoughts?

@BElluu
Copy link

BElluu commented Dec 22, 2019

Hello. Can I take care of this task?

@GaneshSPatil
Copy link
Contributor Author

Hi @BElluu , Thanks for showing interest in picking up this issue.

Currently, we're in the process of finalizing the expected result (how the create entity button should look on each page). Depending upon the operations allowed on the page, the styles may differ slightly. As mentioned in #7473 (comment).

This issue can be picked up once we have a better understanding of the expected result. We'll keep you posted with any further information on this issue.

@BElluu
Copy link

BElluu commented Dec 23, 2019

@GaneshSPatil Thank you so much. Have a peaceful Christmas.

@GaneshSPatil
Copy link
Contributor Author

Hi @gocd/committers,

After having an internal discussion following style has been finalized for the create entity button on the SPAs

Screen Shot 2020-01-09 at 12 09 49 PM

Tasks (Make create/add button consistent on all the following SPAs)

  • Pipelines (/go/admin/pipelines)
  • Environments (/go/admin/environments)
  • Templates (/go/admin/templates)
  • Elastic Agent Configurations (/go/admin/elastic_agent_configurations)
  • Config Repositories (/go/admin/config_repos)
  • Artifact Stores (/go/admin/artifact_stores)
  • Secret Configs (/go/admin/secret_configs)
  • Authorization Configurations (/go/admin/security/auth_configs)
  • Role Configurations (/go/admin/security/roles)

@GaneshSPatil
Copy link
Contributor Author

@BElluu - You can pick up this story. The expected styles for the create entity button and the list of SPAs which require a change has been mentioned #7473 (comment)

Here are some links to get started with GoCD:

  1. Concepts in GoCD
  2. Setting up GoCD development environment

Do let us know if you need any assistance to work on this story.

@BElluu
Copy link

BElluu commented Jan 16, 2020

Hi @GaneshSPatil I'm in! :) This is my first time when I'm trying to contribute. How can I assignee to this task?

@ketan
Copy link
Member

ketan commented Jan 16, 2020

Assigned to @BElluu

@BElluu
Copy link

BElluu commented Jan 19, 2020

Hello!

I have little problem with project on Windows. I installed all requirements for project from https://developer.gocd.org/current/ When I use gradlew clean prepare I got that errors:

This usually happens because your environment has changed since running `npm install`.
Run `npm rebuild node-sass` to download the binding for your current environment.
    at module.exports (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\node-sass\lib\binding.js:15:13)
    at Object.<anonymous> (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\node-sass\lib\index.js:14:35)
    at Module._compile (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\v8-compile-cache\v8-compile-cache.js:192:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
    at Module.load (internal/modules/cjs/loader.js:811:32)
    at Function.Module._load (internal/modules/cjs/loader.js:723:14)
    at Module.require (internal/modules/cjs/loader.js:848:19)
    at require (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\v8-compile-cache\v8-compile-cache.js:161:20)
    at getDefaultSassImplementation (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\sass-loader\dist\getDefaultSassImplementation.js:24:10)
    at getSassImplementation (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\sass-loader\dist\getSassImplementation.js:19:72)
    at Object.loader (E:\prog\gocd\server\src\main\webapp\WEB-INF\rails\node_modules\sass-loader\dist\index.js:40:61)
 @ ./webpack/views/pages/pipeline_configs/stages/jobs/artifacts.scss 2:26-308
 @ ./webpack/views/pages/pipeline_configs/stages/jobs/artifacts_tab_widget.tsx
 @ ./webpack/views/pages/pipeline_configs/spec/stages/jobs/artifacts_tab_widget_spec.tsx
 @ ./webpack sync spec\.(js|msx|ts|tsx)$
 @ ./spec/webpack/specRoot.js

I tried delete node_modules directory and use npm install and npm rebuild node-sass but that does not works.
Below my npm, nodejs and python versions.
image

@GaneshSPatil Should I create next issue for above question or if I ask here, is it fine?

@arvindsv
Copy link
Member

I really don't know much about the setup on Windows, but maybe this is relevant? https://stackoverflow.com/a/41454385

@BElluu
Copy link

BElluu commented Jan 20, 2020

I really don't know much about the setup on Windows, but maybe this is relevant? https://stackoverflow.com/a/41454385

I using Intellij IDEA Community, so I'm not sure that will help but thanks!

@arvindsv
Copy link
Member

Sorry, thought it was some kind of build tool config you need to do (even though you use IntelliJ). Unfortunately, I don't know anyone who develops on GoCD with Windows. Maybe someone else who sees this can help.

@BElluu
Copy link

BElluu commented Jan 20, 2020

@arvindsv so I need to install Linux next to Windows. I tried work on virtual machine with Linux using VirtualBox, but that works so slow ;/

@arvindsv
Copy link
Member

@BElluu Yes, working on VirtualBox will be quite slow. I don't know what the state of Docker on Windows is currently, but I know @tomzo used to use https://github.com/kudulab/docker-gocd-dojo to build GoCD. Not sure it'll work for you on Windows. If not, Linux it is. :)

@BElluu
Copy link

BElluu commented Jan 20, 2020

@arvindsv I tested that on Linux but was too slow. I can't use Docker for Windows because I have Home Edition :( nvm, I need to install Linux next to Windows. I will buy flash drive tomorrow, because I can't find my old :)

@ketan
Copy link
Member

ketan commented Jan 21, 2020

This file lists the specific packages and versions that you would need in order to build/compile gocd. You might be missing the compiler so try this line before installing other packages.

@BElluu
Copy link

BElluu commented Apr 12, 2020

I installed Linux and coz of COVID I have some time soso I try do this task in next days:)

@stale
Copy link

stale bot commented Jul 11, 2020

This issue has been automatically marked as stale because it has not had activity in the last 90 days.
If you can still reproduce this error on the master branch using local development environment or on the latest GoCD Release, please reply with all of the information you have about it in order to keep the issue open.
Thank you for all your contributions.

@stale stale bot added the stale label Jul 11, 2020
@stale stale bot closed this as completed Jul 18, 2020
@chadlwilson chadlwilson added no stalebot Don't mark this stale. and removed stale labels Oct 23, 2021
@chadlwilson chadlwilson reopened this Oct 23, 2021
@chadlwilson chadlwilson removed this from the NextUp milestone Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants