Refactor SocialLink Viewsets

[sh-ran]

Terms

• I have searched all open bug reports
• I agree to follow activist's Code of Conduct

Behavior

This issue would rewrite the SocialLinks ViewSets for Groups, Organizations and Event entities. The current code needs to be re-written to adhere to the current way we are writing our views within the backend. It includes some changes on how the new social links are updated instead of deleting and creating a list within the backend.

For #1289 we'll be centralizing the logic for FAQs such that entities will have instantiated models from central FAQ functionality. We want to do the same for social_links such that organizations, events and groups would all have social link functionality that inherits from central functionality written in the content app.

[andrewtavis]

Thanks for the issue and picking it up, @sh-ran!

[MuhammadNizamani]

@sh-ran are you going to solve this problem

[andrewtavis]

Thanks for your interest in the project, @MuhammadNizamani! Would be best if @sh-ran takes this one as we've had lots of discussions on what needs to happen with this issue. Let us know if there's another issue you'd like to pick up though!

[MuhammadNizamani]

Thanks for your interest in the project, @MuhammadNizamani! Would be best if @sh-ran takes this one as we've had lots of discussions on what needs to happen with this issue. Let us know if there's another issue you'd like to pick up though!

@andrewtavis I'm an experienced Python backend developer. If you come across any issues that require Python, I’d love to help solve them.
I also have experience with Vue.js, but I prefer working with Python.

[andrewtavis]

Would be really really great to work with you, @MuhammadNizamani! Also happy to do a call to onboard you at some point and discuss the project, if you'd like :) Feel free to reach out in our community Matrix chats (you'll see my account to start a direct message in the development and general channels 😊). As far as another issue, ones we could look into are:

• You could assist us finalizing the tests for views and serializes in Expand Views testing for backend #1132 (with @sh-ran) and Expand Serializers testing for backend #1133
• Once the authentication method is finalized there will be some work there to refresh it, but let's hold off on that for now
• There's a nice quality of dev experience issue in Allow for URL to be maintained as a part of backend identifiers in the frontend #1007 as we convert all snake_case in the backend to camelCase in the frontend, but that means that we can't have URL in the frontend as _url goes to Url
• If you want to do some map work, then we'd like a tooltip with the time estimate for arriving in Add tooltip to second map waypoint for time estimations  #894, which could be fun 😊

Let me know if any of the above sounds interesting! Happy to make more suggestions or discuss in a call as mentioned :)

[MuhammadNizamani]

Would be really really great to work with you, @MuhammadNizamani! Also happy to do a call to onboard you at some point and discuss the project, if you'd like :) Feel free to reach out in our community Matrix chats (you'll see my account to start a direct message in the development and general channels 😊). As far as another issue, ones we could look into are:

* You could assist us finalizing the tests for views and serializes in [Expand Views testing for backend #1132](https://github.com/activist-org/activist/issues/1132) (with [@sh-ran](https://github.com/sh-ran)) and [Expand Serializers testing for backend #1133](https://github.com/activist-org/activist/issues/1133)

* Once the authentication method is finalized there will be some work there to refresh it, but let's hold off on that for now

* There's a nice quality of dev experience issue in [Allow for `URL` to be maintained as a part of backend identifiers in the frontend #1007](https://github.com/activist-org/activist/issues/1007) as we convert all `snake_case` in the backend to `camelCase` in the frontend, but that means that we can't have `URL` in the frontend as `_url` goes to `Url`

* If you want to do some map work, then we'd like a tooltip with the time estimate for arriving in [Add tooltip to second map waypoint for time estimations  #894](https://github.com/activist-org/activist/issues/894), which could be fun 😊

Let me know if any of the above sounds interesting! Happy to make more suggestions or discuss in a call as mentioned :)

@andrewtavis I’ve gone through your discussion about writing unit tests, and I’ll start by writing them first — but before that, I’ll review the backend code.

[andrewtavis]

Amazing, @MuhammadNizamani! Feel free to write in the issue you want to work on and we'll get you assigned as well 😊