Page MenuHomePhabricator
Differential D9502

[lib] Remove setCookieOnRequest
ClosedPublic
Actions

Authored by michal on Oct 16 2023, 5:35 AM.
Tags
None
Referenced Files
F2320810: D9502.diff
Tue, Jul 23, 2:53 AM
Unknown Object (File)
Tue, Jul 16, 2:28 PM
Unknown Object (File)
Thu, Jul 4, 3:28 PM
Unknown Object (File)
Sun, Jun 30, 4:24 AM
Unknown Object (File)
Sat, Jun 29, 6:32 PM
Unknown Object (File)
Wed, Jun 26, 3:50 PM
Unknown Object (File)
Mon, Jun 24, 5:55 PM
Unknown Object (File)
Mon, Jun 24, 5:55 PM
View All 41 Files
Subscribers
ashoat
tomek
will

Details

Reviewers
inka
kamil
atul
ashoat
Commits
rCOMMa689d2faedcc: [lib] Remove setCookieOnRequest
Summary

ENG-4747
Depends on D9290

After D9288 both web and native had this config set to true so we can remove it and simplify the code.

Test Plan

yarn flow-all

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

michal created this revision.Oct 16 2023, 5:35 AM
Herald added subscribers: will, tomek, ashoat. · View Herald TranscriptOct 16 2023, 5:35 AM
Harbormaster completed remote builds in B23263: Diff 32036.Oct 16 2023, 5:53 AM
michal requested review of this revision.Oct 16 2023, 5:53 AM
atul accepted this revision.Oct 17 2023, 10:17 AM
atul added a reviewer: ashoat.

Accepting, but adding @ashoat as blocking to take another look

ashoat accepted this revision.Oct 18 2023, 1:41 PM
ashoat added inline comments.
lib/utils/call-server-endpoint.js
78–84 ↗(On Diff #32036)

Thanks for finding / deleting this comment

This revision is now accepted and ready to land.Oct 18 2023, 1:41 PM
Closed by commit rCOMMa689d2faedcc: [lib] Remove setCookieOnRequest (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhyOct 23 2023, 3:05 AM
This revision was automatically updated to reflect the committed changes.
Michal Gniadek <michal.gniadek@swmansion.com> added a commit: rCOMMa689d2faedcc: [lib] Remove setCookieOnRequest.

Revision Contents
Changeset List

PathSize
lib/
utils/
17 lines
1 line
10 lines
native/
1 line
web/
1 line

Diff 32311

View Options

lib/utils/call-server-endpoint.js

Loading...
View Options

lib/utils/config.js

Loading...
View Options

lib/utils/upload-blob.js

Loading...
View Options

native/config.js

Loading...
View Options

web/app.react.js

Loading...