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.

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

Change requests for segments #4044

Open
matthewelwell opened this issue May 29, 2024 · 3 comments · Fixed by #4265
Open

Change requests for segments #4044

matthewelwell opened this issue May 29, 2024 · 3 comments · Fixed by #4265
Assignees
Labels
api Issue related to the REST API feature New feature or request front-end Issue related to the React Front End Dashboard

Comments

@matthewelwell
Copy link
Contributor

Is your feature request related to a problem? Please describe.

When editing segments, I want to be able to request a review from other members of my organisation.

Describe the solution you'd like.

Extend the Change Requests functionality to be able to operate on segments as well as features.

Describe alternatives you've considered

N/a

Additional context

No response

@matthewelwell matthewelwell added feature New feature or request front-end Issue related to the React Front End Dashboard api Issue related to the REST API labels May 29, 2024
@matthewelwell matthewelwell linked a pull request Nov 19, 2024 that will close this issue
@kyle-ssg kyle-ssg reopened this Nov 27, 2024
@kyle-ssg
Copy link
Member

kyle-ssg commented Nov 27, 2024

Feedback from what we found

  • Remove environments from change requests at the project level
  • The Order of change requests seemingly random - should be date descending
  • In order to Store original rule ids so we can compare and diff change requests appropriately, as discussed let's call this field version_of to rules so we have a reference to the original rule that was potentially changed
  • Approve is throwing a 500
    - curl 'https://api-staging.flagsmith.com/api/v1/projects/8500/change-requests/483/approve/' \
    -   -H 'accept: */*' \
    -   -H 'accept-language: en-GB,en;q=0.9' \
    -   -H 'authorization: Token e163db1b90f7d08f024800558c134464ed6ef2c2' \
    -   -H 'content-type: application/json' \
    -   -H 'origin: http://localhost:8080' \
    -   -H 'priority: u=1, i' \
    -   -H 'referer: http://localhost:8080/' \
    -   -H 'sec-ch-ua: "Google Chrome";v="131", "Chromium";v="131", "Not_A Brand";v="24"' \
    -   -H 'sec-ch-ua-mobile: ?0' \
    -   -H 'sec-ch-ua-platform: "macOS"' \
    -   -H 'sec-fetch-dest: empty' \
    -   -H 'sec-fetch-mode: cors' \
    -   -H 'sec-fetch-site: cross-site' \
    -   -H 'user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/131.0.0.0 
Safari/537.36' \
    -   --data-raw '{}'
  • Project level change requests aren't storing the user that created it, this is currently working for environment change requests
  • Fix email link from /projects/x/change-requests to project/x/change-requests
  • Manage project level change requests permission - The description should be clarified to be "Ability to create, delete and publish change requests associated with a project".

@zachaysan
Copy link
Contributor

Regarding the the order of change requests seemingly random, I set them to paginate based on id not date descending since that is what most of the backend orders by for other endpoints. It should be functionally the same as created at.

@zachaysan
Copy link
Contributor

Approve is throwing a 500

This was solved by fixing the user being set on create. The bug was in the email method and a missing user blew the stack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Issue related to the REST API feature New feature or request front-end Issue related to the React Front End Dashboard
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants