Page MenuHomePhabricator
Differential D9502

[lib] Remove setCookieOnRequest
ClosedPublic
Actions

Authored by michal on Oct 16 2023, 5:35 AM.
Tags
None
Referenced Files
F3542037: D9502.id32036.diff
Thu, Dec 26, 8:08 AM
Unknown Object (File)
Wed, Dec 25, 8:12 AM
Unknown Object (File)
Fri, Dec 13, 10:47 PM
Unknown Object (File)
Nov 7 2024, 11:46 PM
Unknown Object (File)
Nov 7 2024, 11:38 PM
Unknown Object (File)
Nov 7 2024, 11:37 PM
Unknown Object (File)
Nov 7 2024, 7:31 PM
Unknown Object (File)
Oct 4 2024, 12:52 PM
View All 66 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...