Skip to content

visibility check#3493

Closed
prakhar-singh1928 wants to merge 0 commit intomainfrom
dev_admin_visiblity_fix
Closed

visibility check#3493
prakhar-singh1928 wants to merge 0 commit intomainfrom
dev_admin_visiblity_fix

Conversation

@prakhar-singh1928
Copy link
Collaborator

🔗 Related Issue

Closes #


📝 Summary

What does this PR do and why?


🏷️ Type of Change

  • Bug fix
  • Feature / Enhancement
  • Documentation
  • Refactor
  • Chore (deps, CI, tooling)
  • Other (describe below)

🧪 Verification

Check Command Status
Lint suite make lint
Unit tests make test
Coverage ≥ 80% make coverage

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • Tests added/updated for changes
  • Documentation updated (if applicable)
  • No secrets or credentials committed

📓 Notes (optional)

Screenshots, design decisions, or additional context.

@prakhar-singh1928 prakhar-singh1928 marked this pull request as draft March 5, 2026 09:02
@prakhar-singh1928 prakhar-singh1928 changed the title Temp fix for admin bypass fix for admin bypass Mar 5, 2026
@prakhar-singh1928 prakhar-singh1928 changed the title fix for admin bypass visibility check Mar 5, 2026
@crivetimihai crivetimihai self-assigned this Mar 7, 2026
@prakhar-singh1928 prakhar-singh1928 force-pushed the dev_admin_visiblity_fix branch from d16ec80 to 8c60078 Compare March 19, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants