Skip to content

Commit

Permalink
Remove Approver users
Browse files Browse the repository at this point in the history
Due to limitations on the DSI public API, we've had to re-evaluate some of bits of this work
and at the moment displaying the list of users in the school or their approvers does not seem
like much value.

This fix addresses it by:
- Removing approver users route, controller and index view
- Removing organisation_users
- Removing specs for `organisation_users`
  • Loading branch information
ekumachidi committed Mar 27, 2024
1 parent 97a5d6c commit bbef7c9
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 168 deletions.
9 changes: 0 additions & 9 deletions app/controllers/schools/approvers_controller.rb

This file was deleted.

29 changes: 0 additions & 29 deletions app/services/schools/dfe_sign_in_api/organisation_users.rb

This file was deleted.

53 changes: 0 additions & 53 deletions app/views/schools/approvers/index.html.erb

This file was deleted.

1 change: 0 additions & 1 deletion config/routes.rb
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@

resource :change_school, only: %i[show create], as: 'change', path: 'change', controller: 'change_schools'
resources :users, only: %i[index new create show], controller: 'users'
resources :approvers, only: %i[index], controller: 'approvers'
resource :prepopulate_school_profiles, only: %i[create]

resource :dashboard, only: :show
Expand Down
76 changes: 0 additions & 76 deletions spec/services/schools/dfe_sign_in_api/organisation_users_spec.rb

This file was deleted.

0 comments on commit bbef7c9

Please sign in to comment.