Skip to content
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

Closed
wants to merge 2 commits into from

Conversation

Baig-fatema
Copy link

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

  • API Integration: Implemented a connection to [API Name], allowing for the retrieval of high-quality images relevant to the content on the website.
  • Dynamic Image Loading: Images are now fetched dynamically, ensuring that users see fresh and diverse content each time they visit the site.
  • Responsive Design: Adjusted the layout to ensure that images are displayed correctly across various screen sizes, enhancing accessibility for all users.
  • Error Handling: Added error handling to manage API failures gracefully, providing fallback content when necessary.

Benefits

  • Improved User Engagement: By offering a wider variety of images, users are more likely to engage with the content, making their experience more enriching.
  • Enhanced Aesthetic Appeal: The integration adds a modern touch to the site, making it visually attractive and in line with current design trends.
  • Support for Future Expansion: This implementation lays the groundwork for further enhancements, such as user-uploaded images or additional image sources.

Testing

  • Thoroughly tested the image loading functionality across different devices and browsers to ensure compatibility and responsiveness.
  • Verified that fallback mechanisms work correctly in cases where the API fails to respond.

Next Steps

  • Review and feedback are welcome! Any suggestions for improvement or additional features are appreciated as we strive to make SeriniFi the best resource for anxiety relief.

Thank you! @Amna-Hassan04 for considering this enhancement!

Copy link

@github-actions github-actions bot left a 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:

  1. After completing a task, add your name and details to the Contributors Table/List.
  2. Attach relevant screenshots or videos of your contribution for transparency.
  3. 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.
  4. Propose New Ideas: Discuss new features or suggestions in the "Ideas" channel before starting work.

@Amna-Hassan04
Copy link
Owner

@Baig-fatema Please attach a screen recording of the working feature

@Baig-fatema
Copy link
Author

@Amna-Hassan04 this is the link to screen recording features of the #63.

video link: https://drive.google.com/file/d/132RnjYjTDJzTAdpXcVPJtv3-KyOmHArW/view

  • i am requesting that please review and merge this.
  • this images are responsive and dynamic.
  • copy the above link and paste in chrome browser so that you can review it.

@Baig-fatema
Copy link
Author

@Amna-Hassan04 also i am requesting that please add the gssoc label and add my points in leader board.

@Baig-fatema
Copy link
Author

@Amna-Hassan04 and @dev-shinde i am requesting please review it. and please merge if possible and add my points in leader board.

@Amna-Hassan04
Copy link
Owner

@Baig-fatema it shows this
image

@dev-shinde
Copy link
Collaborator

@Baig-fatema give the video view access to all ; so that we can review it

@Baig-fatema
Copy link
Author

Copy link
Owner

@Amna-Hassan04 Amna-Hassan04 left a 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.

@zoobiasyed
Copy link
Collaborator

@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.

@Amna-Hassan04
Copy link
Owner

No Reply from more than 10 days hence closing pr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhance Image Display with API Integration
4 participants