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

Jump to bottom

UI: Broken audit log line for updated pipelines #2302

Closed
ZJvandeWeg opened this issue Jun 16, 2023 · 2 comments · Fixed by #3124
Closed

UI: Broken audit log line for updated pipelines #2302

ZJvandeWeg opened this issue Jun 16, 2023 · 2 comments · Fixed by #3124
Assignees
@Steve-Mcl
Labels
area:frontend For any issues that require work in the frontend/UI bug Something isn't working size:XS - 1 Sizing estimation point task A piece of work that isn't necessarily tied to a specific Epic or Story.

Comments

@ZJvandeWeg
Copy link
Member

Description

The status line doesn't display human friendly text, nor an icon.

Screenshot 2023-06-16 at 11 42 53

Epic/Story

No response

Have you provided an initial effort estimate for this issue?

I have provided an initial effort estimate
@ZJvandeWeg ZJvandeWeg added bug Something isn't working task A piece of work that isn't necessarily tied to a specific Epic or Story. labels Jun 16, 2023
@joepavitt joepavitt added area:frontend For any issues that require work in the frontend/UI size:XS - 1 Sizing estimation point labels Jun 16, 2023
@Steve-Mcl Steve-Mcl self-assigned this Nov 24, 2023
Steve-Mcl added a commit that referenced this issue Nov 24, 2023
@Steve-Mcl
Add UI for pipeline update audit entry
b24dff0 
fixes #2302
@Steve-Mcl Steve-Mcl mentioned this issue Nov 24, 2023
Merged
11 tasks
@Steve-Mcl
Copy link
Contributor

Question for @ZJvandeWeg and @joepavitt

Forgetting for the moment, all new audit entries should have the UI when they are submitted, do we want to consider presenting new Audit entries a little friendlier?

For example, by default, remove the : , replace the dot.s with a space, then apply Sentence case or Title Case?

Which would make : application.pipeline.updated appear as Application Pipeline Updated

@joepavitt
Copy link
Contributor

As a default backup - yes, although that won't cover the "icon" side of things, but it is better

@Steve-Mcl Steve-Mcl mentioned this issue Nov 24, 2023
Closed
@Steve-Mcl Steve-Mcl mentioned this issue Dec 22, 2023
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Steve-Mcl Steve-Mcl

Labels
area:frontend For any issues that require work in the frontend/UI bug Something isn't working size:XS - 1 Sizing estimation point task A piece of work that isn't necessarily tied to a specific Epic or Story.
Projects
☁️ Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add UI for pipeline update audit entry FlowFuse/flowfuse
3 participants
@ZJvandeWeg @Steve-Mcl @joepavitt