Skip to content

fix: delete project#3393

Merged
dabeeeenster merged 1 commit intomainfrom
fix/delete-project
Feb 8, 2024
Merged

fix: delete project#3393
dabeeeenster merged 1 commit intomainfrom
fix/delete-project

Conversation

@kyle-ssg
Copy link
Copy Markdown
Member

@kyle-ssg kyle-ssg commented Feb 7, 2024

Thanks for submitting a PR! Please check the boxes below:

  • I have run pre-commit to check linting
  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Deleting a project redirects you to the project list immediately and shows a toast message when it is deleted. This prevents a bunch of permission errors.

How did you test this code?

  • Deleted a project

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2024 9:32am
flagsmith-frontend-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2024 9:32am
flagsmith-frontend-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2024 9:32am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 7, 2024

Uffizzi Preview deployment-46042 was deleted.

@dabeeeenster dabeeeenster added this pull request to the merge queue Feb 8, 2024
Merged via the queue into main with commit be544e2 Feb 8, 2024
@dabeeeenster dabeeeenster deleted the fix/delete-project branch February 8, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

front-end Issue related to the React Front End Dashboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deleting a project shows 'Oops!' permission error

3 participants