CodeForAfrica / CodeForAfrica/academy.AFRICA

[Audit][P1] Consolidate account verification and activation state

Open
#56 0 comments 0 reactions 1 assignee Claimed by @koechkevin View on GitHub
bug
Dominant language
PHP
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

## Context

Account verification is enforced without a migration for existing users, while a second legacy account-status system remains active. The two paths use different metadata and activation formats.

Audit findings: 6, 7, 10, and 14.

## Scope

- Define one authoritative account state and activation flow.
- Migrate or safely grandfather existing users before enforcement.
- Remove the legacy `key`/`user_id`, `account_status`, and hardcoded `wp_users` path.
- Make login, session, social-auth, and API behavior consistent.

## Done when

- Existing legitimate and privileged users are not locked out.
- New unverified accounts are handled consistently.
- Activation tokens are validated before state changes.
- Login, REST, XML-RPC, and application-password behavior has integration coverage.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.