Page MenuHomePhabricator

[lib] Add client side boilerplate and reducer for an endpoint that allows creating or updating public links
ClosedPublic

Authored by tomek on May 19 2023, 10:00 AM.
Tags
None
Referenced Files
F1550288: D7879.diff
Mon, Apr 15, 3:46 PM
Unknown Object (File)
Fri, Apr 12, 6:30 PM
Unknown Object (File)
Wed, Apr 10, 4:15 AM
Unknown Object (File)
Mon, Apr 8, 11:44 AM
Unknown Object (File)
Fri, Apr 5, 4:22 AM
Unknown Object (File)
Fri, Apr 5, 4:22 AM
Unknown Object (File)
Fri, Apr 5, 4:22 AM
Unknown Object (File)
Fri, Apr 5, 4:22 AM
Subscribers

Details

Summary

The only interesting part is that the link in redux is updated after successful call to this endpoint. For that purpose I'm using a link returned by backend, instead of figuring out the values on the client, because at some point we could consider having different defaults.

Depends on D7878

Test Plan

Tested in combination with the rest of the stack. Checking if updating or creating a public link works is enough to verify the correctness.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil added inline comments.
lib/actions/link-actions.js
67–75 ↗(On Diff #26682)

why not return response;?

This revision is now accepted and ready to land.May 22 2023, 2:41 AM
lib/actions/link-actions.js
67–75 ↗(On Diff #26682)

The response may contain more fields and we should make sure that all the other fields are filtered out.