Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Sponsor dashboard, reject/hide application of provider #1787

Closed
wants to merge 3 commits into from
Closed

Sponsor dashboard, reject/hide application of provider #1787

wants to merge 3 commits into from

Conversation

aghimpu
Copy link
Contributor

@aghimpu aghimpu commented May 17, 2021

@aghimpu aghimpu requested a review from dev-rminds May 17, 2021 04:02
@maxvisser
Copy link
Contributor

maxvisser commented May 21, 2021 •
edited
Loading

image
Seems that you added a whole new row; You should only add the delete (bin) button

@maxvisser maxvisser self-requested a review May 21, 2021 10:31
maxvisser
maxvisser suggested changes May 21, 2021
View reviewed changes
Copy link
Contributor

@maxvisser maxvisser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove extra row and move the bin button the existing row

@maxvisser maxvisser assigned aghimpu and unassigned dev-rminds May 25, 2021
@aghimpu aghimpu requested a review from maxvisser May 27, 2021 20:35
@maxvisser
Copy link
Contributor

@dev-rminds lets discuss this during next refinement whats still needed to merge

@lexlog
Copy link
Contributor

lexlog commented Apr 20, 2022

refinement note - remove it due to not clear enough proposal

@lexlog lexlog closed this Apr 20, 2022
@lexlog lexlog deleted the feature-reject-provider-application branch August 25, 2023 08:34
@lexlog lexlog restored the feature-reject-provider-application branch August 25, 2023 08:35
@lexlog lexlog deleted the feature-reject-provider-application branch August 25, 2023 08:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dev-rminds dev-rminds Awaiting requested review from dev-rminds

@maxvisser maxvisser Awaiting requested review from maxvisser

Assignees

@aghimpu aghimpu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aghimpu @maxvisser @lexlog @dev-rminds