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

feat(3197): Table cell component for stages in pipeline list jobs view #1230

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

tkyi
Copy link
Member

@tkyi tkyi commented Nov 10, 2024

Context

The new jobs UI makes use of custom components for each cell in the table. This creates a new component for handling the stage name for the job.

Objective

Create a glimmer component for the jobs.
V1:
Screenshot 2024-11-09 at 10 03 13 PM

V2:
Screenshot 2024-11-09 at 10 05 27 PM

References

screwdriver-cd/screwdriver#3197
screwdriver-cd/screwdriver#3200

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

import { computed } from '@ember/object';

export default Component.extend({
stageName: computed('record.stage.stageName', {
Copy link
Member

Choose a reason for hiding this comment

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

Locally I got an error when viewing something without a stage.
Screenshot 2024-11-12 at 4 47 49 PM

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

After some investigation, this issue is caused when a pipeline has jobs that never run before, the change to the jobs list view to include the stages is not able to handle this at the moment.

To reproduce: create any new pipeline and don't run any builds, check into the jobs list view and console.

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.

2 participants