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

fix:issue on do not validate locale if not creating for all locales #19799

Merged

Conversation

binar1
Copy link
Contributor

@binar1 binar1 commented Mar 16, 2024

What does it do?

After applying a change this pull request some cases has been missed like for example when the local is not default one and you want to create and publish

Why is it needed?

Because the user can not create or save any single type module so i just add another validation to check if the local exist or not

How to test it?

Create single types and collection type with localization enabled , look to you database you can see all data will be create depend of your current local and when you are going to publish it it says 404 because it can not find the record with that local in the database

Related issue(s)/PR(s)

Copy link

vercel bot commented Mar 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
contributor-docs ❌ Failed (Inspect) Mar 27, 2024 5:40am

@strapi-cla
Copy link

strapi-cla commented Mar 16, 2024

CLA assistant check
All committers have signed the CLA.

Marc-Roig
Marc-Roig previously approved these changes Mar 25, 2024
Copy link
Contributor

@Marc-Roig Marc-Roig left a comment

Choose a reason for hiding this comment

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

thank you for the fix 🚀

@Marc-Roig Marc-Roig self-assigned this Mar 25, 2024
@Marc-Roig Marc-Roig added this to the 4.21.1 milestone Mar 25, 2024
@Marc-Roig Marc-Roig added source: plugin:i18n Source is plugin/i18n package fix:4 Fix made in Strapi 4 labels Mar 25, 2024
@joshuaellis joshuaellis added the pr: fix This PR is fixing a bug label Mar 25, 2024
@Marc-Roig
Copy link
Contributor

@binar1 could you run prettier to format the file you updated? 🙏 (yarn prettier:code)

@binar1
Copy link
Contributor Author

binar1 commented Mar 25, 2024

Hello @Marc-Roig noted. I've pushed a new commit where Prettier has been used.

@Marc-Roig
Copy link
Contributor

Marc-Roig commented Mar 25, 2024

will run the CI and once its green will merge this work, did a quick check and seems to be working, but we will run it through a QA process during the week and release this on 4.21.1 . Thank you again!

@Marc-Roig
Copy link
Contributor

Looks like some tests are failing, I am taking a look now

@Marc-Roig Marc-Roig dismissed their stale review March 25, 2024 16:47

there are some fixes to be done

@Marc-Roig Marc-Roig added this to the 4.22.0 milestone Mar 27, 2024
@Marc-Roig Marc-Roig merged commit 4fa6d95 into strapi:develop Mar 27, 2024
94 of 95 checks passed
@Marc-Roig
Copy link
Contributor

Marc-Roig commented Mar 27, 2024

This work will be finally released in next week 4.22.0 release, great job 🚀

@nevosol
Copy link

nevosol commented Apr 4, 2024

Great job on fixing the issue, thank you!

@strapi-bot
Copy link

This pull request has been mentioned on Strapi Community Forum. There might be relevant details there:

https://forum.strapi.io/t/translating-single-type-returns-404-and-then-500/37345/4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community fix:4 Fix made in Strapi 4 pr: fix This PR is fixing a bug source: plugin:i18n Source is plugin/i18n package
Projects
Status: Fixed/Shipped
Development

Successfully merging this pull request may close these issues.

None yet

8 participants