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

Add validation for response body && api key check #681

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

7empestx
Copy link

@7empestx 7empestx commented Apr 1, 2024

This PR addresses a critical issue with the GitHub contributions widget, which was found to be malfunctioning. The initial diagnosis led to the setup of a local development environment, during which it was discovered that the Unsplash API key was not configured. This oversight resulted in an unexpected response from the fetch request, subsequently triggering an error during the data mapping process due to the response body being of an unanticipated format.

The underlying issue has been identified as a missing Unsplash API key which, when not set, leads to a chain of uncaught exceptions culminating in an obscure error message. The necessary error handling has been implemented to ensure that such cases are handled gracefully in the future, prompting the developer to set the API key.

Additionally, this PR serves as an inquiry into the current status and future plans for the GitHub contribution calendar widget. We have noticed potential discrepancies in the API response from https://api.bloggify.net/gh-calendar/?username=7empestx, which might indicate changes in the API endpoint behavior or output. Clarification on this would be beneficial for further troubleshooting and subsequent rectification of the widget functionality.

Attached below is a visual representation of the widget issue for reference:

image

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

Successfully merging this pull request may close these issues.

None yet

1 participant