NoMongo: Postgres Migration Support for Advertisement in orgDashboard

[gautam-divyanshu]

Is your feature request related to a problem? Please describe.
Update query and mutations according to the recently migrated develop-postgres branch of talawa-api.

Describe the solution you'd like
Make the schema support the changes

Currently no advertisements in develop-postgres

Expected:

Potential internship candidates

Please read this if you are planning to apply for a Palisadoes Foundation internship

• Student Internship Programs talawa#359

[yashaghane21]

@palisadoes @gautam-divyanshu can i work on this ?

[Bittukr7479]

@gautam-divyanshu I want to work on this issue please assign me.

[mpdesai90]

Our policy is to assign no more than two issues to each contributor across all repositories. This way everyone gets a chance to participate in the projects. We sometimes give exceptions for more urgent cases and sometimes we lose track, but the policy stands. You have reached your limit, please wait until your existing issues are closed before requesting more issues. You could unassign yourself from one of the other issues too.

[Bittukr7479]

Our policy is to assign no more than two issues to each contributor across all repositories. This way everyone gets a chance to participate in the projects. We sometimes give exceptions for more urgent cases and sometimes we lose track, but the policy stands. You have reached your limit, please wait until your existing issues are closed before requesting more issues. You could unassign yourself from one of the other issues too.

Ok sir, don't worry.

[udayempire]

Can I work on this sir?

[udayempire]

Also Just to clarify I am not wrong we want to make sure that that we are able to make advertisments in the server using this -

[palisadoes]

Yes

[udayempire]

Yes

Please assign me I would like to work on it.

[palisadoes]

This issue aims to restore functionality to the app after migrating to a PostgreSQL API back end.

1. This is a high priority issue. It's a prerequisite for our GSoC ideas.
2. The develop branches of Talawa Admin and Talawa API work together. Use those branches to determine the expected behavior of the apps.
3. Compare the operation of Talawa Admin and Talawa API running together in the develop-postgres branch.
4. Submit your PR against the develop-postgres branch and attach a video.

[JaiPannu-IITI]

@palisadoes If no one is working on this, I'd like to take it.

[udayempire]

@JaiPannu-IITI I am working actually had college exams so I was busy for past few days will Update the work soon since exams are over.

[JaiPannu-IITI]

Perfect, take your time !!

[udayempire]

@palisadoes @gautam-divyanshu while I was solving this issue to get all the advertisments to be shown in frontend .I Think we need to have a change in talawa-api in advertisments.ts here we have query to get only 1 advertisment at a time.

we need to have a query with findMany to fetch all the advertisments.

similarly in graphql.schema too we need to add a query here

since this is issue is to work on talawa-admin codebase but since we need to update talawa-api too should I open an issue there?

Please let me know if I am lacking something.Looking forward thanks

[udayempire]

Also the possible solution I see to show active and completed advertisements on this page is that first we fetch all the advertisements for a particular organisation from the backend and filter them as active or completed in frontend and show the data.

[palisadoes]

Thanks.

1. Please open an issue to update the API
2. It'll be best to filter by organization and date range by default with options for further filtering by the active/completed status

[udayempire]

Sure I will look into it.

[udayempire]

@palisadoes Sir,I have completed around 70% of the work and would really appreciate your feedback.

advertismentWork.1.mp4

Things that are still pending:

1. Media Attachments: I haven't implemented the functionality to attach media images yet.
2. Auto-Scrolling Ads: When the limit is set to 6 ads, new ads do not automatically scroll into view.
3. Frontend Refresh Issue: Changes in advertisements do not reflect instantly on the frontend; a manual page refresh is required to see updates.

[palisadoes]

How's progress on this? It needs to be completed as a GSoC prerequisite.

[udayempire]

Everything is done I am just failing some test cases in 3 files in test files.

[bandhan-majumder]

@udayempire are you still working on this? Can I help you fix?

[udayempire]

@bandhan-majumder i will be glad if u can currently the issue is with failing test cases ...rest functionality wise its working

[bandhan-majumder]

Sure @udayempire . It will be helpful for me if you make sure that there are no conflicts in the existing pr so that while failing in the test cases in the workflow check, I can utilized it for changes needed.

[udayempire]

Sure @udayempire . It will be helpful for me if you make sure that there are no conflicts in the existing pr so that while failing in the test cases in the workflow check, I can utilized it for changes needed.

Sure

[udayempire]

Sure @udayempire . It will be helpful for me if you make sure that there are no conflicts in the existing pr so that while failing in the test cases in the workflow check, I can utilized it for changes needed.

Hi sorry for being late you can check the pr as there arent any conflicting pr

[bandhan-majumder]

@udayempire the errors are causing because there are multiple imports of the same things in a single file and some things are not imported at all but used in the file. I have added comments in 2 such areas. The errors are pretty self explanatory. Did you push the code from CLI? Our pre-commit hook should already prevent this though. Let me know if you need more info about it.

[udayempire]

But I havent updated the code in organization file then why it is coming?

[udayempire]

But I havent updated the code in organization file then why it is coming?its just the pulled code.

[udayempire]

Also can you check the 3 files i mentioned I removed all the test errors in entry.spec just 1-2 is there..

[bandhan-majumder]

check slack dm.

[udayempire]

we have discussed and @bandhan-majumder can fix the next issues please assign him this issue

[bandhan-majumder]

Thanks @udayempire. @palisadoes @Cioppolo14 please assign. I will fix this ASAP.

[bandhan-majumder]

@palisadoes

Should one be able to upload multiple attachments? Currently it's supporting a single one. If so, how the multiple attachments will be displayed? In a sliding manner from left to right?

Also, updating media files are not supported til now. Should I add them in api?

[palisadoes]

Yes to both questions

[bandhan-majumder]

@palisadoes Alright.

[bandhan-majumder]

@palisadoes

update:

basic functionalities are recovered

https://www.loom.com/share/8f83dc4a7f6a4dfa8e7e879476f00844

working on optimization and tests for now.