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 #4228

[mlehotskylf]

Blocker issue #4228 is in progress.

[mlehotskylf]

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

[mlehotskylf]

https://github.com/communitybridge/easycla/issues/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:

• https://github.com/sun-test-org/newTestRepo01/pull/3

  user details

• GitHub: thakurveerendras1

[mlehotskylf]

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