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

The blue box (message) should not be displayed when an analyst or superuser accessed the domain via the admin. #2899

Closed
vickyszuchin opened this issue Oct 4, 2024 · 1 comment · Fixed by #3094

Comments

@vickyszuchin
Copy link

Current Behavior

An analyst or superuser looking at a portfolio domain may be confused by message "you don't have access to manage..." (see slack thread) when they have permission to edit ("edit" link is shown).

Expected Behavior

The blue box should not be displayed when an analyst or superuser accessed the domain via the admin.

Steps to Reproduce

  1. Analyst views portfolio domain via the admin (/admin)
  2. Blue box with "you don't have access to manage..."
  3. User is allowed to edit domain information

Environment

No response

Additional Context

Slack thread with screenshot

Issue Links

No response

@vickyszuchin vickyszuchin added dev bug Something that isn't working as intended labels Oct 4, 2024
@abroddrick abroddrick moved this from 👶 New to 🍦 Backlog in .gov Product Board Oct 7, 2024
@abroddrick abroddrick removed the bug Something that isn't working as intended label Oct 10, 2024
@abroddrick
Copy link
Contributor

Removing the bug tag, this was meant to be a feature change not a bug. We never designed it so that this box wouldn't appear, bugs are reserved for things that are not working as expected

@abroddrick abroddrick moved this from 🍦 Backlog to 🎯 Ready in .gov Product Board Oct 10, 2024
@dave-kennedy-ecs dave-kennedy-ecs self-assigned this Nov 12, 2024
@vickyszuchin vickyszuchin moved this from 🎯 Ready to 🔖 Planned in .gov Product Board Nov 12, 2024
@dave-kennedy-ecs dave-kennedy-ecs moved this from 🔖 Planned to 🍦 Backlog in .gov Product Board Nov 15, 2024
@dave-kennedy-ecs dave-kennedy-ecs moved this from 🍦 Backlog to 🏗 In progress in .gov Product Board Nov 15, 2024
@abroddrick abroddrick moved this from 🏗 In progress to 👀 In review in .gov Product Board Nov 20, 2024
dave-kennedy-ecs added a commit that referenced this issue Nov 20, 2024
#2899: don't show blue box when analyst clicks 'Manage Domain'
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in .gov Product Board Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

4 participants