Skip to content
New issue

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

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

Already on GitHub? # to your account

[RAD-5674] Stallion - Creating a script to change the Role XID’s #19

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

francisco-cruz-o
Copy link
Contributor

@francisco-cruz-o francisco-cruz-o commented Mar 5, 2025

Description

RAD-5674
The issue involves creating a script to change Role XIDs in the Stallion Prod/Dev systems due to an error preventing the modification of these XIDs.

A/C

  • The script should correct Role XIDs containing a space with a hyphen or underscore.
  • Data linked to the changed XIDs must maintain their connection to the updated Roles.

Current behavior

  • Role XID has spaces

Expected behavior

  • Role XID without spaces

Changes

  • create script to update role XID

Resources

Captura de pantalla 2025-03-05 a la(s) 10 38 31 a  m Captura de pantalla 2025-03-05 a la(s) 10 40 34 a  m

Tests

  • Manual testing

@terrypacker
Copy link
Contributor

@francisco-cruz-o yes there are some risks to consider:

  1. I would be using the service to update the role
  2. When you change the XIDs of the role it shouldn't affect permissions since these are all FKs to the IDs in the database but I would setup some tests on a system to confirm this.
  3. There are probably some edge cases that you will run into where a role is stored in the data column of a VO which may not get updated. Another example is if someone inserted roles into the JSON store as an object, these won't get changed. I think the UI settings may do this.

@terrypacker
Copy link
Contributor

If you want to setup some time for me to walk you through testing it in the Mango UI go ahead and grab a slot.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants