Skip to content

Add ".js" extension to imports in fluent-react #577

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

Merged
merged 1 commit into from
Oct 25, 2021

Conversation

indeyets
Copy link
Contributor

Similar change was applied to different modules, but react was skipped somehow. Lack of this change breaks build on webpack5

@gregtatum gregtatum requested a review from eemeli September 22, 2021 16:01
Copy link
Member

@eemeli eemeli left a comment

Choose a reason for hiding this comment

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

Yeah, this looks like a pretty clear case of oversight about applying that change to this file.

@indeyets
Copy link
Contributor Author

indeyets commented Oct 7, 2021

@eemeli looks like github-actions need explicit approval for external contributions. Also, can you cut the release with these changes please?

Similar change was applied to different modules, but react was skipped somehow. Lack of this change breaks build on webpack5
@MichaelKohler
Copy link

@eemeli what's the status on this?

@eemeli eemeli merged commit 3526567 into projectfluent:master Oct 25, 2021
@eemeli
Copy link
Member

eemeli commented Oct 25, 2021

The status is that this fell somewhere between the todo-lists of @gregtatum and myself, along with a healthy dose of travelling in the interim.

In other words, apologies for not getting to this sooner.

@indeyets indeyets deleted the patch-1 branch October 25, 2021 08:11
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.

3 participants