Skip to content
This repository was archived by the owner on May 30, 2024. It is now read-only.

Conversation

@noahingh
Copy link
Member

@noahingh noahingh commented Oct 7, 2021

In the /approvals API, it blocks to create a new approval with the deployer. In UI, Not to display the deployer in the candidates of approval.

@noahingh noahingh added the bug Something isn't working label Oct 7, 2021
@noahingh noahingh added this to the First Release milestone Oct 7, 2021
@noahingh noahingh merged commit f243811 into main Oct 7, 2021
@noahingh noahingh deleted the fix-approval-list branch October 7, 2021 13:40
})
.addCase(searchCandidates.fulfilled, (state, action) => {
state.candidates = action.payload
state.candidates = action.payload.filter(candidate => candidate.id !== state.deployment?.deployer?.id)
Copy link
Member Author

Choose a reason for hiding this comment

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

Not to display the deployer in candidates.

})
.addCase(searchCandidates.fulfilled, (state, action) => {
state.candidates = action.payload
state.candidates = action.payload.filter(candidate => (candidate.id !== state.user?.id))
Copy link
Member Author

Choose a reason for hiding this comment

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

Not to display the user who will deploy in candidates.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants