kobotoolbox / kpi / 28599329881 / 5

81%

master: 76%

push

github

refactor(projectSettings): tsify DEV-2214 (#7190)

### 🗒️ Checklist

1. [x] run linter locally
2. [x] update developer docs (API, README, inline, etc.), if any
3. [x] for user-facing doc changes create a Zulip thread at `#Support
Docs Updates`, if any
4. [x] draft PR with a title `<type>(<scope>)<!>: <title> DEV-1234`
5. [x] assign yourself, tag PR: at least `Front end` and/or `Back end`
or `workflow`
6. [x] fill in the template below and delete template comments
7. [x] review thyself: read the diff and repro the preview as written
8. [x] open PR & confirm that CI passes & request reviewers, if needed
9. [x] act on any greptile review below a 5/5 score or leave comment
explaining why you won't
10. [ ] delete this checklist section from the final squash commit
before merging

### 💭 Notes

Changes here:
- **projectSettings.tsx**: Migrated from JavaScript to TypeScript
  - Added interfaces
  - Tried to keep type casting minimal (no `any` 👍)
- Dynamic field assignment now uses a switch statement to avoid casting
`as any`
  
- **mixins.tsx**: Added type definitions for the legacy mixins (we're
still using react-mixin for now)
  - Created DmixProps, DmixState, and DmixContext interfaces
- All mixin methods have explicit `this` parameter typing so TypeScript
understands what props/state are available at runtime
  - Both contextRouter and dmix mixins now have full type signatures

No functional changes - everything works the same, just with
compile-time type safety.

### 👀 Preview steps

1. ℹ️ Have a project
2. Open the project and go to Settings tab
3. Try editing different fields and saving:
   - Project name
   - Description  
   - Sector/Country
4. 🟢 Everything should save normally
5. If you have a deployed project, try archiving and unarchiving it from
settings
6. 🟢 Should work exactly like the main branch

Nothing should break since this is (almost) purely internal TypeScript
changes.

---------

Co-authored-by: James Kiger <68701146+jamesrkiger@users.n... (continued)

4540 of 12456 branches covered (36.45%)

18863 of 37017 relevant lines covered (50.96%)

0.51 hits per line