-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhance Image Display with API Integration #63 #89
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better.
Please follow these guidelines for smooth collaboration:
- After completing a task, add your name and details to the Contributors Table/List.
- Attach relevant screenshots or videos of your contribution for transparency.
- Tasks are assigned on a first-come, first-served basis.
- Avoid spamming requests for already-assigned tasks.
- Respect the queue and wait for new tasks.
- Propose New Ideas: Discuss new features or suggestions in the "Ideas" channel before starting work.
@Baig-fatema Please attach a screen recording of the working feature |
@Amna-Hassan04 this is the link to screen recording features of the #63. video link: https://drive.google.com/file/d/132RnjYjTDJzTAdpXcVPJtv3-KyOmHArW/view
|
@Amna-Hassan04 also i am requesting that please add the gssoc label and add my points in leader board. |
@Amna-Hassan04 and @dev-shinde i am requesting please review it. and please merge if possible and add my points in leader board. |
@Baig-fatema it shows this |
@Baig-fatema give the video view access to all ; so that we can review it |
@Amna-Hassan04 and @dev-shinde Sorry for inconvineince. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem is that I dont think the images in this api are visually appealing, since the image rigth now on our home page is very cozy it relaxes u but the images in the implemented api are very dull, more anxeity triggering, so as this is a app whose focus is to have mental wellness this should be made better.
@Baig-fatema your pr will only be merged if all the relevant details and pictures/videos are mentioned. Also all the test cases should be passed, before the deadline. |
No Reply from more than 10 days hence closing pr. |
Pull Request Description: Enhance Image Display with API Integration
Overview
This pull request addresses issue #63, which aims to enhance the image display functionality on the SeriniFi website by integrating an external image API. The changes made in this PR improve the user experience by providing a more dynamic and visually appealing presentation of images related to anxiety relief.
Changes Made
Benefits
Testing
Next Steps
Thank you! @Amna-Hassan04 for considering this enhancement!