question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Placeholder issue for new Pipeline Group SPA

See original GitHub issue
Issue Type
  • Bug Report
Summary

Placeholder issue for new Pipeline Group SPA

  • When group admin tries to delete a pipeline group, GET calls are made to endpoints api/admin/security/roles and api/users which returns 403 response. This causes generic error (Failed to retrieve pipelines) shown to user. User need to refresh the page to load the pipeline groups.

  • Pipeline group permissions create/edit failure messages are not shown completely. Say an admin user tried to update the pipeline group permissions. But provides a non existing role name. On save the API returns appropriate errors Role <some_role> does not exists. But UI shows just a generic error message as Validations failed for pipelines 'GROUP1'. Error(s): [Validation failed.]. Please correct and resubmit.

Issue Analytics

  • State:closed
  • Created 4 years ago
  • Comments:5 (5 by maintainers)

github_iconTop GitHub Comments

1reaction
GaneshSPatilcommented, Jan 8, 2020

@kritika-singh3 - IMO, I would go for option 1 instead of updating the API. As this is just for the UI representation(In perspective GoCD authorization, other two permissions does not have any meaning when a user has admin permission on pipeline group)

Wouldn’t this lead to having different representations for the config? An API call says [view, admin] and UI says [view, operate, admin]??

IMO, our SPAs should be dumb, they should just represent whatever the API sends as is!

1reaction
bdpipravacommented, Jan 8, 2020

@kritika-singh3 - IMO, I would go for option 1 instead of updating the API. As this is just for the UI representation(In perspective GoCD authorization, other two permissions does not have any meaning when a user has admin permission on pipeline group)

Read more comments on GitHub >

github_iconTop Results From Across the Web

[Pipeline Config SPA] [Placeholder] Issues with new ... - GitHub
This is a placeholder issue to document issues with in-development pipeline config SPA. Tabs ready for QA All Tabs are ready for QA...
Read more >
Add new placeholders for group/project badges - GitLab.org
I want to be able to link a coverage badge to a report that is published with gitlab pages. Unfortunately, there is no...
Read more >
Advanced: Placeholders | Jira Service Management Data ...
Set/update an object attribute with the value from an issue field when issue is transitioned to a specific status.
Read more >
Stantec
Our team is interested in new technologies and market trends in order to provide our clients with comprehensive engineering services in terms of...
Read more >
BASF – United States
BASF strengthens innovation pipeline for sustainable agriculture. BASF strengthens its activities in research and development (R&D) for sustainable agricultural ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found