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
F13313949: D21846.diff
Tue, Jun 11, 2:58 PM
F13307520: D21846.id52067.diff
Sun, Jun 9, 6:59 PM
F13305720: D21846.id52068.diff
Sun, Jun 9, 12:12 AM
F13293032: D21846.diff
Wed, Jun 5, 6:26 AM
F13270464: D21846.id52068.diff
Wed, May 29, 9:44 AM
F13270463: D21846.id52067.diff
Wed, May 29, 9:44 AM
F13270462: D21846.id.diff
Wed, May 29, 9:44 AM
F13252090: D21846.diff
Sat, May 25, 1:04 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 52068

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