Page MenuHomePhabricator
Differential D9502

[lib] Remove setCookieOnRequest
ClosedPublic
Actions

Authored by michal on Oct 16 2023, 5:35 AM.
Tags
None
Referenced Files
F6234977: D9502.id32311.diff
Thu, Apr 24, 1:55 PM
F6230630: D9502.id32036.diff
Thu, Apr 24, 10:03 AM
Unknown Object (File)
Thu, Apr 24, 7:18 AM
Unknown Object (File)
Tue, Apr 22, 9:05 PM
Unknown Object (File)
Wed, Apr 9, 1:03 AM
Unknown Object (File)
Tue, Apr 8, 10:52 PM
Unknown Object (File)
Tue, Apr 8, 10:38 PM
Unknown Object (File)
Sun, Apr 6, 5:39 AM
View All 97 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

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

lib/utils/config.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

lib/utils/upload-blob.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

native/config.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

web/app.react.js

Loading...