Page MenuHomePhabricator
Differential D21846

Update Slowvote voting methods to use sensible string constants
ClosedPublic
Actions

Authored by epriestley on May 26 2022, 5:24 PM.
Tags
None
Referenced Files
F18081810: D21846.diff
Tue, Aug 5, 2:23 AM
F18026490: D21846.diff
Sat, Aug 2, 10:08 PM
F17844001: D21846.diff
Sun, Jul 27, 1:34 AM
F17674431: D21846.diff
Sun, Jul 13, 7:16 AM
F17657484: D21846.diff
Sat, Jul 12, 12:18 PM
Unknown Object (File)
Jun 29 2025, 12:34 AM
Unknown Object (File)
Jun 27 2025, 8:09 PM
Unknown Object (File)
May 29 2025, 6:02 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13682: Improve Remarkup file reference attachment behaviors after removal of "attach by default"
Commits
rP03d3d1889d60: Update Slowvote voting methods to use sensible string constants
Summary

Ref T13682. Use API-friendly string constants instead of opaque integers in Slowvote voting methods.

Test Plan

Created, edited, and voted in polls with various voting methods. Examined database after migrations.

Diff Detail

Repository
rP Phabricator
Branch
slowvote6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25754
Build 35582: arc lint + arc unit

Event Timeline

epriestley created this revision.May 26 2022, 5:24 PM
Owners added a subscriber: Restricted Owners Package.May 26 2022, 5:24 PM
epriestley requested review of this revision.May 26 2022, 5:24 PM
Harbormaster completed remote builds in B25754: Diff 52067.May 26 2022, 5:24 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 26 2022, 5:27 PM
Closed by commit rP03d3d1889d60: Update Slowvote voting methods to use sensible string constants (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP03d3d1889d60: Update Slowvote voting methods to use sensible string constants.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
5 lines
src/
applications/
slowvote/
constants/
4 lines
controller/
2 lines
storage/
2 lines
view/
6 lines

Diff 52067

View Options

resources/sql/autopatches/20220525.slowvote.06.method-type.sql

Loading...
View Options

resources/sql/autopatches/20220525.slowvote.07.method-value.sql

Loading...
View Options

src/applications/slowvote/constants/SlowvotePollVotingMethod.php

Loading...
View Options

src/applications/slowvote/controller/PhabricatorSlowvoteEditController.php

Loading...
View Options

src/applications/slowvote/storage/PhabricatorSlowvotePoll.php

Loading...
View Options

src/applications/slowvote/view/SlowvoteEmbedView.php

Loading...
Phabricator · Built by Phacility