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

Screenshots not saving #203

Closed
simonhamp opened this issue Apr 21, 2022 · 7 comments · Fixed by #205
Closed

Screenshots not saving #203

simonhamp opened this issue Apr 21, 2022 · 7 comments · Fixed by #205
Assignees

Comments

@simonhamp
Copy link

I've removed some old screenshots for my package and attempted to upload new ones. They appear to upload, but when I hit 'save' they don't appear.

Is there some approval process they need to go through?

@marcusmoore
Copy link
Contributor

Hi @simonhamp. There isn't an approval process so you're hitting a bug. Thanks for reporting. I'll investigate tomorrow 😄

@marcusmoore
Copy link
Contributor

Thanks for reporting this @simonhamp. I was able to get it fixed and also manually associated the screenshots you uploaded previously to your package 😄

@simonhamp
Copy link
Author

Legend! Thank you

@mziraki
Copy link

mziraki commented Aug 9, 2022

@marcusmoore this bug still exists!

@marcusmoore
Copy link
Contributor

Thanks for reporting @mziraki. This is fixed (again).

I see you have multiple packages and I don't want to manually associate screenshots to the wrong package so please feel free to re-upload them.

Thanks again 😄

@mziraki
Copy link

mziraki commented Aug 9, 2022

@marcusmoore thanks, I noticed one minor mistake in screenshots url
image
and I think there are lots of unlinked screenshots in storage, I don't know if there is any regular purge schedule, but it will be useful if it existed.

@marcusmoore
Copy link
Contributor

Thanks, I noticed that as well and it should be a quick fix I'm planning on tackling tomorrow 👍🏾

We have a scheduled task that cleans up abandoned screenshots too 😄

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 a pull request may close this issue.

3 participants