Page MenuHomePhabricator
Differential D9502

[lib] Remove setCookieOnRequest
ClosedPublic
Actions

Authored by michal on Oct 16 2023, 5:35 AM.
Tags
None
Referenced Files
F2624095: D9502.diff
Sat, Sep 7, 12:21 AM
Unknown Object (File)
Thu, Sep 5, 8:18 PM
Unknown Object (File)
Thu, Aug 29, 4:38 AM
Unknown Object (File)
Sun, Aug 25, 2:38 AM
Unknown Object (File)
Tue, Aug 13, 1:56 PM
Unknown Object (File)
Aug 6 2024, 9:21 PM
Unknown Object (File)
Aug 6 2024, 9:21 PM
Unknown Object (File)
Aug 6 2024, 9:20 PM
View All 51 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
No Lint Coverage
Unit
No Test Coverage

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

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 32036

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...