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

Update path-too-long-exception.md #6569

Merged
merged 3 commits into from
Oct 22, 2024
Merged

Conversation

Abdjulaziz
Copy link
Member

Description

Changed Database query

What did you add/update/change?
Changed database query to show media items longer than 80 characters. Previous query showed all media now you only get the media you need to edit.

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Deadline (if relevant)

When should the content be published?
As soon as possible

Changed database query to show media items longer than 80 characters. Previous query showed all media now you only get the media you need to edit.
@sofietoft
Copy link
Contributor

Thanks for the PR @Abdjulaziz ! Makes sense to change this query 💪

Reading the steps that comes after running the query.. would it make sense to update those as well?
They talk about identifying the files with the long name 🤔 I guess that won't be necessary with this new script?

What do you think?

@Abdjulaziz
Copy link
Member Author

I'll look into that and fix it 👍🏻

Just noticed there's a little bit more that can be improved

smal edit - no further changes needed
@Abdjulaziz
Copy link
Member Author

@sofietoft had to test something and ask internally to see if the steps were correct and they were so I did a small change to the text and removed some things

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

Looks great @Abdjulaziz ! 💪

I just have a single tiny suggestion.

@Abdjulaziz
Copy link
Member Author

Sound good

@sofietoft
Copy link
Contributor

Thanks again for the PR @Abdjulaziz ! 💪

I'll get it merged.

@sofietoft sofietoft merged commit 5d854b0 into umbraco:main Oct 22, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants