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

[BUG] Screenshots error shows multiple times #4612

Closed
zateutsch opened this issue Jan 16, 2024 · 0 comments
Closed

[BUG] Screenshots error shows multiple times #4612

zateutsch opened this issue Jan 16, 2024 · 0 comments
Assignees
@zateutsch
Labels
bug 🐛 manifest-editor site

Comments

@zateutsch
Copy link
Contributor

What happened?

Screenshot error will display multiple times in Manifest Editor.

image

How do we reproduce the behavior?

  1. Edit manifest with no screenshots.
  2. Use manifest editor.
  3. Screenshot errors will display repeatedly.

What do you expect to happen?

Error shows only once.

What environment were you using?

OS: Windows 11
Browser: edge

Additional context

I'll take this.
@zateutsch zateutsch self-assigned this Jan 16, 2024
@zateutsch zateutsch mentioned this issue Jan 16, 2024
Merged
@zateutsch zateutsch moved this to In Review 🤔 in PWABuilder Jan 16, 2024
github-merge-queue bot pushed a commit that referenced this issue Jan 17, 2024
@zateutsch @jgw96
#4172/4610/4612: Manifest Editor bug fixes (#4615)
5daf5d1 
fixes #4172
fixes #4610 
fixes #4612

Fixed a few bugs with how errors are displayed in the manifest editor.

I had to change the manifest tests a little to get this to work as
expected, so @jgw96 or @Jaylyn-Barbee one of y'all should probably take
a close look at this PR to make sure I didn't cause anything unexpected.

---------

Co-authored-by: Justin Willis <jgw9617@gmail.com>
@zateutsch zateutsch moved this from In Review 🤔 to Done ✔️ in PWABuilder Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zateutsch zateutsch

Labels
bug 🐛 manifest-editor site
Projects
PWABuilder
Status: Done ✔️
Milestone
No milestone
Development

No branches or pull requests
2 participants
@beth-panx @zateutsch