chore: refactor rateskill code in viewholders

[batbrain7]

Fixes #2269

Changes: Added the API call at only one place which is the MessageViewHolder which is extended by other viewholders

[ci-reporter]

The build is failing

✨ Good work on this PR so far! ✨ Unfortunately, the Circle CI build is failing as of 7a7c949. Here's the output:

Run Tests

> Task :app:preBuild UP-TO-DATE

I'm sure you can fix it! If you need help, don't hesitate to ask a maintainer of the project!

---

Failed build for cdd2535

spotless check

> Task :app:spotlessKotlin FAILED

Failed build for 8c8e8c4

spotless check

Failed build for 8c8e8c4

spotless check

---

This comment was automagically generated by ci-reporter. If you see a problem, open an issue here.

[atm1504]

@batbrain7 the circle ci probably failed because of heavy load in the circleci server. Just retrigger it. It will be able to build.

[batbrain7]

@iamareebjamal I am able to successfully run the tests locally, could you please assert is triggering the CI will help ?

[liveHarshit]

Just retrigger it. It will be able to build.

I don't think so, IMO need to do changes in unit test files.

[atm1504]

Just retrigger it. It will be able to build.

I don't think so, IMO need to do changes in unit test files.

Probably u r right. But, I faced this issue earlier and retriggering it did work out. If error lies in the unit test file, I think we should make an issue to solve it. : )

[atm1504]

@iamareebjamal I am able to successfully run the tests locally, could you please assert is triggering the CI will help ?

@batbrain7 just add or delete an unnecessary line and push it. It will automatically trigger the build. : )

[liveHarshit]

@batbrain7 just add or delete an unnecessary line and push it. It will automatically trigger the build. : )

Oh, what a great idea... 😅

[batbrain7]

all checks have been passed