greenlight icon indicating copy to clipboard operation
greenlight copied to clipboard

Access codes rectification 6: Restricted unsupervised updates to access codes.

Open KH-Amir-TN opened this issue 2 years ago • 3 comments

Description

Rectifying the rooms access codes to become a room setting that can be configured like any other setting.

This PR completes 6.

~~1. Create and populate glViewerAccessCode and glModeratorAccessCode in PopulateRoomsConfigurations and PopulateMeetingOptions.~~ ~~2. Remove :viewer_access_code and :moderator_access_code migrations.~~ ~~3. Create Room#viewer_access_code, Room#moderator_access_code, Room#generate_viewer_access_code, Room#generate_moderator_access_code, Room#remove_viewer_access_code, Room#remove_moderator_access_code.~~ ~~4. Update RoomsController (#generate_access_code, #remove_access_code) and tests (meeting_controller_spec and rooms_controller_spec).~~ ~~5. Update and extend model spec room_spec.~~ ~~6. Update RoomSettings#update API to restrict edit to codes.~~

Testing Steps

  1. Pull the code.
  2. Install the dependencies bundle install && npm|yarn install.
  3. Clean the previous assets build by running rm app/assets/builds/* (This won't remove .keep since it's hidden).
  4. Clean the database and tmp files for a better isolation by running rails tmp:clear && rails db:schema:cache:clear && rails db:drop && rails db:create && rails db:migrate:with_data
  5. Run the linter and specs bundle exec rubocop --parallel && bundle exec rspec && npx eslint app/javascript/* --ext .jsx,.js
  6. Run ./bin/dev to run the assets builders processes and the Puma server all at once.

Screenshots (if appropriate):

KH-Amir-TN avatar Aug 07 '22 12:08 KH-Amir-TN

~~DEPENDS ON #3714~~

KH-Amir-TN avatar Aug 07 '22 12:08 KH-Amir-TN

REBASED ON e35a8820c41bdf1560c8e65438e8ccae4c415bc2

KH-Amir-TN avatar Aug 15 '22 17:08 KH-Amir-TN

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Aug 15 '22 17:08 sonarqubecloud[bot]