react-native-sse icon indicating copy to clipboard operation
react-native-sse copied to clipboard

Fix: Disable cache control and x formatted by header

Open arpitmalik opened this issue 1 year ago • 14 comments
trafficstars

arpitmalik avatar May 03 '24 11:05 arpitmalik

@wojciechkrol @EmilJunker Thank you for the patience. Please review now.

arpitmalik avatar Jun 03 '24 11:06 arpitmalik

@wojciechkrol @EmilJunker Any update?

arpitmalik avatar Jun 06 '24 12:06 arpitmalik

@arpitmalik Your changes look good to me, although it's a bit difficult to review because of all the unrelated formatting changes you made. I must say that I'm not a big fan of stretching simple if conditions and function calls over multiple lines of code.

EmilJunker avatar Jun 06 '24 13:06 EmilJunker

Please merge the code if it looks good. Formatting is done only to make the code readable.

arpitmalik avatar Jun 11 '24 11:06 arpitmalik

Please merge the code if it looks good. Formatting is done only to make the code readable.

I guess formatting was automatically made by code editor plugins like prettier? I suggest leave these unrelated formatting untouched and obey the one-line condition rule on newly added lines as the author said, everyone have a different opinion on readable code, and you may not want him spend extra time on restoring code to how it looked before.

greenlihui avatar Jun 13 '24 08:06 greenlihui

@EmilJunker @wojciechkrol Please merge now.

arpitmalik avatar Oct 03 '24 08:10 arpitmalik

LGTM 👍

@wojciechkrol Can you please merge this as well as #63. Both of these pull requests would make great additions to this library.

EmilJunker avatar Oct 03 '24 10:10 EmilJunker

@wojciechkrol Can we please merge this on priority? Please check.

arpitmalik avatar Oct 14 '24 05:10 arpitmalik

@wojciechkrol Can you please merge both these on priority. Its been more than 3 months.

DeepikaSharma5 avatar Nov 12 '24 06:11 DeepikaSharma5

@wojciechkrol Reminder^

arpitmalik avatar Nov 18 '24 09:11 arpitmalik

@wojciechkrol Please merge

arpitmalik avatar Dec 02 '24 07:12 arpitmalik

@wojciechkrol Please be active on this. Merge this pull request as soon as possible.

arpitmalik avatar Dec 18 '24 11:12 arpitmalik

@wojciechkrol Please merge

arpitmalik avatar Feb 25 '25 11:02 arpitmalik