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.

Sign up for GitHub

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

Jump to bottom

Various bugfixes #340

Merged
merged 4 commits into from
Apr 12, 2021
Merged

Various bugfixes #340

merged 4 commits into from
Apr 12, 2021

Conversation

ErikIDX
Copy link
Contributor

@ErikIDX ErikIDX commented Apr 12, 2021 •
edited
Loading

  • Fixes IDX API Generic error throwing a notice for the REST API
    • IDX API class uses the status code 'Generic' for unknown errors, need to convert that to an actual HTTP status code for the REST API.
  • Fixes block editor error with invalid API key
  • Fixes Social Pro class failing to create post

@ErikIDX ErikIDX changed the title Impb 1296 bugfixes Fixes IDX API Generic error throwing a notice for the REST API and fixes block editor error with invalid API key Apr 12, 2021
@ErikIDX
Copy link
Contributor Author

ErikIDX commented Apr 12, 2021

Also fixes issue #337 as it was needed for testing

@ErikIDX ErikIDX changed the title Fixes IDX API Generic error throwing a notice for the REST API and fixes block editor error with invalid API key Various bugfixes Apr 12, 2021
@ErikIDX ErikIDX merged commit 17d0f78 into idxbroker:release/3.0.0 Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgreenl5 sgreenl5 sgreenl5 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ErikIDX @sgreenl5