Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move Secrets to Settings #20

Closed
12 of 19 tasks
taitus opened this issue Apr 3, 2019 · 1 comment
Closed
12 of 19 tasks

Move Secrets to Settings #20

taitus opened this issue Apr 3, 2019 · 1 comment
Assignees
@taitus
Labels
Module 6 reviewing

Comments

@taitus
Copy link
Member

taitus commented Apr 3, 2019 •
edited
Loading

Why is this Needed?

To make it easier for an administrator to configure the Consul application from the administrator panel without requiring a systems administrator

Description

Move to structure defined on #19 the next valid Secrets (config/secrets.yml) :

  • rollbar_server_token
  • http_basic_auth
  • http_basic_username
  • http_basic_password
  • twitter_key
  • twitter_secret
  • facebook_key
  • facebook_secret
  • google_oauth2_key
  • google_oauth2_secret
  • census_api_end_point
  • census_api_institution_code
  • census_api_portal_name
  • census_api_user_code
  • sms_end_point
  • sms_username
  • sms_password
  • map_tiles_provider
  • map_tiles_provider_attribution

Secrets discarded by its dependencies with Ansible:

  • secret_key_base
  • server_name

Secrets discarded because they are specific to Madrid City Council:

  • managers_url
  • managers_application_key

Current Behavior

All these variables are included on config/secrets.yml

Expected Behavior

In these screenshots we can see how structure all commented secrets:
https://drive.google.com/drive/folders/1TUWyzQ-YgXjkueV3jq0kP0wGFS3egRHq?usp=sharing
@taitus taitus mentioned this issue Apr 3, 2019
Open
3 tasks
@taitus taitus added the Module 6 label Apr 3, 2019
@taitus taitus mentioned this issue Apr 4, 2019
Open
@taitus taitus mentioned this issue Jul 17, 2019
Closed
This was referenced Jul 29, 2019
Closed
Closed
@taitus taitus removed the in progress label Sep 3, 2019
This was referenced Sep 4, 2019
Merged
Merged
Merged
@taitus taitus mentioned this issue Oct 5, 2019
Merged
@taitus
Copy link
Member Author

taitus commented Oct 11, 2019

Close via #43 #45 #46 #55

@taitus taitus closed this as completed Oct 11, 2019
taitus pushed a commit that referenced this issue Nov 17, 2020
@decabeza @taitus
Improve styles for budgets voting phase (#20)
f3d34a4 
Improve styles for budgets voting phase
taitus pushed a commit that referenced this issue Nov 26, 2020
@decabeza @taitus
Improve styles for budgets voting phase (#20)
16fd294 
Improve styles for budgets voting phase
taitus pushed a commit that referenced this issue Nov 26, 2020
@decabeza @taitus
Improve styles for budgets voting phase (#20)
093bcfa 
Improve styles for budgets voting phase
taitus pushed a commit that referenced this issue Dec 2, 2020
@decabeza @taitus
Improve styles for budgets voting phase (#20)
498df57 
Improve styles for budgets voting phase
Senen added a commit that referenced this issue Jun 29, 2021
@Senen
Merge pull request #20 from AyuntamientoLorca/mimemagic
8a969fb 
Bump mimemagic to version 0.3.10
taitus pushed a commit that referenced this issue Jan 27, 2022
@tvararu @tsundokul
Change user validation (#20)
1d6fcb1 
* #18 Partial implementation for user verification

* #18 Add validation for CNP

* #18 Add working implementation for validation

* #18 Fix unsafe stripping

* Add test for CNP validation, split up terms checkboxes

Add a very basic system spec for the new CNP verification flow.

Split up the checkbox to accept terms, residency, and adult status into
three separate checkboxes.

Co-authored-by: tsundokul <[email protected]>
Senen pushed a commit that referenced this issue May 26, 2023
@javierm
Merge pull request 'Upgrade to Consul 1.5.0' (#20) from upgrade_to_1.…
b19422c 
…5.0 into master

Reviewed-on: https://git.nubarron.teide.int/shs-consul/cabildoabierto-consul/pulls/20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@taitus taitus

Labels
Module 6 reviewing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Senen @taitus