Contributor status not updated while PR automatic covered after add user under approval list on ecla without required ICLA case

[thakurveerendras]

Summary

Contributor status not updated while PR automatically covered after adding user under the approval list on ecla without the required ICLA case

Steps

1. Add cla group with not selected " Require corporate contributors to sign Individual CLAs "
2. Add PR which CLA gated
3. Go to the company cla page & add a GitHub user under the approval list

Actual results

User added under contributor list with "Not Authorized" status

Expected behavior

User should be added under the contributor list with "Authorized" status

Screenshots

[umeshlumbhani247]

@thakurveerendras
Working fine on my side, Can you please remove duplicate signatures if those are exists. May be due to duplicate signatures, this issue may occurs.
CC: @mlehotskylf

[mlehotskylf]

@thakurveerendras can you please confirmed this has been tested? If yes then please change the status. Thanks!

[thakurveerendras]

Created new setup & testing is blocked due to the below issue

• EasyCLA: User added under approval list not displayed under approval list at corporate console EasyCLA: User added under approval list not display under approval list at corporate console  #4228

[mlehotskylf]

Blocker issue #4228 is in progress.

[mlehotskylf]

#4228 is not fixed yet but @thakurveerendras can complete testing of this fix.

[mlehotskylf]

#4228 is also in QA Ready now.

[thakurveerendras]

Issue is still reproducible as reported on the dev site,

[nickmango]

@thakurveerendras could you give more details on this ? The cla-group in question . Did it have icla and ccla configured ?
In addition what was the PR ? I need to be able to reproduce this

[thakurveerendras]

Kindly refer to the requested details

• Yes, ICAL & CCLA configured on cla

Project details:

• Name: Venus
• https://lfx.dev.platform.linuxfoundation.org/foundation/a09P000000DsCE5IAN/project/a09P000000DsCEtIAN/cla

PR:

• Update README.md sun-test-org/newTestRepo01#3

user details

• GitHub: thakurveerendras1

[mlehotskylf]

This is corner case and lower priority so we can fix it in next release.