Skip to content

Conversation

@cc-matthias-m
Copy link

Hi,

With the last version of react-native (0.75.4), the package display a warning for a deprecated code :

 ERROR  Warning: A props object containing a "key" prop is being spread into JSX:
  let props = {key: someKey, indicator: ..., style: ..., source: ..., accessible: ..., accessibilityLabel: ...};
  <FitImage {...props} />
React keys must be passed directly to JSX without using spread:
  let props = {indicator: ..., style: ..., source: ..., accessible: ..., accessibilityLabel: ...};
  <FitImage key={someKey} {...props} />

So I create this PR to fix it. As you can see, it’s really a light commit.

This PR work with last react-native version and previous one.

Do you think create a new npm patch 0.0.23 to add this modification ?

Thanks for reading.

@ericpoulinnz
Copy link

@obipawan can this PR be merged in and a new release pushed out? Getting the same issue with React Native 0.76

@rafatrace
Copy link

Can we merge this and get a new version please?

1 similar comment
@RoiDayan1
Copy link

Can we merge this and get a new version please?

@lehresman
Copy link

We're running into this too. Can we get a new release with this PR merged?

@rejas
Copy link

rejas commented Nov 11, 2025

I am not sure if @obipawan still is interested in maintaining this repo (or has the time), although we and lots others find it helpful. So first: thanks to them for this library!

I took the liberty of forking this repo and applying @cc-matthias-m PR on top of it (so we can use it in our app)

Depending upon the original creator we might continue work on that (https://github.com/Volksverpetzer/react-native-hyperlink) or (if they want) take over this repo.

Any thoughts or comments to this?

@cc-matthias-m
Copy link
Author

I am not sure if @obipawan still is interested in maintaining this repo (or has the time), although we and lots others find it helpful. So first: thanks to them for this library!

I took the liberty of forking this repo and applying @cc-matthias-m PR on top of it (so we can use it in our app)

Depending upon the original creator we might continue work on that (https://github.com/Volksverpetzer/react-native-hyperlink) or (if they want) take over this repo.

Any thoughts or comments to this?

That sounds good.
Since we haven't heard from the creator in over a year, we can create a new repository to maintain this package.

The first person who has time should:

  • Create a new repository
  • Update the README:
    • Explain the situation and indicate that this is the new maintained repository for the package
    • Update the "Installation" section to reflect the new package name
  • Publish the package on npm

Thanks to whoever takes this on!

@rejas
Copy link

rejas commented Nov 12, 2025

We at Volksverpetzer would be happy to maintain this, the repo is already forked and available at https://github.com/Volksverpetzer/react-native-hyperlink

Big question to start with though: What name should the new fork have to distunguish itself in npm etc? Any ideas from you @cc-matthias-m ?

We could scope this with our name like @volksverpetzer/react-native-hyperlink or @vvp/react-native-hyperlink...

@cc-matthias-m
Copy link
Author

Generally, the convention is to prefix the package name with the author/company name.

So @volksverpetzer/react-native-hyperlink is a good solution.

@rejas
Copy link

rejas commented Nov 15, 2025

Generally, the convention is to prefix the package name with the author/company name.

So @volksverpetzer/react-native-hyperlink is a good solution.

I just published v0.0.23 under https://www.npmjs.com/package/@volksverpetzer/react-native-hyperlink

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.

6 participants