This is a preview of the identities support.
This will allow to link several identities to an user account,
and to switch from one to another.
Fixes T476.
Details
Details
- Reviewers
- None
- Maniphest Tasks
- T476: Implement subaccounts
Diff Detail
Diff Detail
- Repository
- rGROVE Auth Grove
- Lint
No Lint Coverage - Unit
No Test Coverage - Branch
- identities
- Build Status
Buildable 12 Build 12: arc lint + arc unit
Event Timeline
Comment Actions
@Sandlayth is interested to test this feature, but not comfortable with the idea to have to review PHP code.
| app/Identity.php | ||
|---|---|---|
| 8 | 4 spaces? | |
| app/Models/User.php | ||
|---|---|---|
| 151 ↗ | (On Diff #1100) | According getCurrentIdentity: // Tries to autoselect identity if there is only one // or if one is configured to be used by default. Only the first case is here implemented, not the default one. |
| composer.json | ||
| 10–11 | Can probably be switched to rKOT? | |
| database/factories/ModelFactory.php | ||
| 25 | use Keruald\OmniTools\Identifiers\UUID; 'uuid' => UUID::UUIDv4(); Or we can also use the new Random::generateIdentifier() to provide 20 bytes instead of 16, but a more compact string ? | |
| database/migrations/2015_07_12_000000_create_identities_table.php | ||
| 17 | Can't this be the primary key? | |