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

fix: issues in permission in config-api #66

Closed
duttarnab opened this issue Feb 14, 2022 · 1 comment
Closed

fix: issues in permission in config-api #66

duttarnab opened this issue Feb 14, 2022 · 1 comment
Assignees
@mjatin-dev
Labels
bug Something isn't working comp-admin-ui Component affected by issue or PR

Comments

@duttarnab
Copy link
Contributor

duttarnab commented Feb 14, 2022 •
edited
Loading

Describe the bug

  • A row is moved to the last row of the table when a description is added to it. Ideally, it should at the same place
  1. Add description to the first row record
  2. After save it is automatically moved to the last row of table.
  • The heading of the Permission page is Config API Roles. It should be Config API Permissions.

image

  • No information appears when we expand a row.
    image
@duttarnab duttarnab added bug Something isn't working comp-admin-ui Component affected by issue or PR labels Feb 14, 2022
syntrydy added a commit that referenced this issue Feb 14, 2022
@syntrydy
feat(admin-ui): unable to add description of an existing permission i…
806f4fe 
…n config-api #66
@syntrydy syntrydy mentioned this issue Feb 14, 2022
Merged
@mo-auto mo-auto mentioned this issue Feb 15, 2022
Merged
@duttarnab duttarnab assigned mjatin-dev and unassigned syntrydy Feb 17, 2022
@duttarnab duttarnab changed the title fix: unable to add description of an existing permission in config-api fix: issues in permission in config-api Feb 17, 2022
@syntrydy
Copy link
Contributor

I think i fixed this while working on another task

@mo-auto mo-auto mentioned this issue Mar 15, 2022
Merged
This was referenced Jul 14, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mjatin-dev mjatin-dev

Labels
bug Something isn't working comp-admin-ui Component affected by issue or PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@syntrydy @mjatin-dev @duttarnab