Page MenuHomePhabricator

[native] Rename RegistrationButton
ClosedPublic

Authored by inka on Aug 20 2024, 12:33 AM.
Tags
None
Referenced Files
F2758610: D13118.id43471.diff
Thu, Sep 19, 4:08 AM
Unknown Object (File)
Tue, Sep 10, 7:51 AM
Unknown Object (File)
Mon, Sep 9, 7:35 AM
Unknown Object (File)
Sat, Sep 7, 3:25 PM
Unknown Object (File)
Thu, Sep 5, 8:33 PM
Unknown Object (File)
Thu, Sep 5, 2:32 PM
Unknown Object (File)
Thu, Sep 5, 2:16 PM
Unknown Object (File)
Thu, Sep 5, 5:49 AM
Subscribers

Details

Summary

Addressing review from D13080

Test Plan

flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Aug 20 2024, 12:50 AM
tomek added inline comments.
native/components/loading-button.react.js
14 ↗(On Diff #43471)

I'm wondering if this name can be improved. Why is it called loading? Maybe we can make this less specific and call it primary? What do you think? Does it make sense?

This revision is now accepted and ready to land.Aug 20 2024, 1:24 AM
This revision was automatically updated to reflect the committed changes.