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

Fixed Plural Formatting For Newer Babel Versions #5

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

TheKruspe
Copy link
Contributor

No description provided.

@TheKruspe TheKruspe mentioned this pull request Aug 23, 2022
@macnewbold
Copy link

Thanks for this! We've been running into the same error lately and were looking into a fix. It's always nice when someone else has already made the pull request you were about to write!

@imacat
Copy link

imacat commented Jan 2, 2023

Thanks. It works. I hope this can be merged into the main branch and a new release will be out soon. ^_^

@Zbysekz
Copy link

Zbysekz commented Oct 4, 2023

Hello, would be nice to merge this, it fixes the problem

@julienbordet
Copy link

Had the same problem, was on the verge of fixing it and saw @TheKruspe did the job. Thanks to him for that.
@emdemir , could you please merge it ?

@BornSupercharged
Copy link

BornSupercharged commented Dec 30, 2024

No movement on getting this fixed merged in for this long, come on @emdemir please?

In the meantime, I forked the repo and made the fix, and updated my requirements.txt as such:
Flask-Babel-JS @ git+https://github.com/BornSupercharged/Flask-Babel-JS@master
( replacing Flask-Babel-Js==1.0.2 )

It now works properly.

@emdemir
Copy link
Owner

emdemir commented Dec 30, 2024

Sorry for not responding for so long. I wrote this project a few jobs ago and never had a use for it afterwards. I'll probably end up rewriting it since embedding the JS file doesn't make much sense. I'm not even sure if it works on the latest Flask version.

@BornSupercharged
Copy link

BornSupercharged commented Dec 30, 2024

@emdemir if you could approve the PR and release tag 1.0.3 that would let everyone just use Flask-Babel-Js==1.0.3

Unless I'm going crazy and 1.0.1 was the latest; in which case just release it as 1.0.2

Even if you're going to rewrite it. Please consider approving and making a new tag for this fix, it's been verified by many different people over the past 2 years and it's literally just one line of code.

FWIW: Flask-Babe-JS is still working with Flask 3.1.0 as of today.

Thank you!

@emdemir
Copy link
Owner

emdemir commented Dec 30, 2024

FWIW: Flask-Babe-JS is still working with Flask 3.1.0 as of today.

That's impressive ^^; I'll take a look at this after New Year's. Allow me two more days please.

@emdemir
Copy link
Owner

emdemir commented Dec 30, 2024

Actually, looking at the issue, this seems good to merge away. I'll release a new version as soon as I locate my PyPI credentials...

@emdemir emdemir merged commit a33c138 into emdemir:master Dec 30, 2024
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 this pull request may close these issues.

7 participants