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.

Sign up for GitHub

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

Jump to bottom

added access filter for registered persone #989

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

added access filter for registered persone #989

wants to merge 2 commits into from

Conversation

DimaDiD
Copy link
Contributor

@DimaDiD DimaDiD commented Jul 1, 2022

dev

JIRA

Code reviewers

  • @github_username

Second Level Review

  • @github_username

Summary of issue

ToDo

Summary of change

ToDo

Testing approach

ToDo

CHECK LIST

  • СI passed
  • Сode coverage >=95%
  • PR is reviewed manually again (to make sure you have 100% ready code)
  • All reviewers agreed to merge the PR
  • I've checked new feature as logged in and logged out user if needed
  • PR meets all conventions

@DimaDiD DimaDiD requested review from tvardero and vsynieok July 1, 2022 11:03
@DimaDiD DimaDiD self-assigned this Jul 1, 2022
@Serhii-Buniak Serhii-Buniak self-requested a review July 1, 2022 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tvardero tvardero tvardero approved these changes

@vsynieok vsynieok vsynieok approved these changes

@Serhii-Buniak Serhii-Buniak Serhii-Buniak approved these changes

At least 3 approving reviews are required to merge this pull request.

Assignees

@DimaDiD DimaDiD

Labels
User Profile
Milestone
No milestone