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

bbPress - make sure the functions to process image uploads are all there #132

Closed

Conversation

foliovision
Copy link

Description of the Change

When user tries to submit the image on the bbPress profile editing page, it fails to process. Some functions are missing and user gets a white screen.

Alternate Designs

None.

Possible Drawbacks

None.

Verification Process

bbPress user profile editing now saves properly when the image is selected for avatar.

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Changelog Entry

Fixed - image upload on bbPress profile edit page

Credits

Props @foliovision

@faisal-alvi
Copy link
Member

faisal-alvi commented May 27, 2022

Hi @foliovision thanks for raising a PR. However, I'm wondering, exactly where are you trying to upload an image? can you please share steps to regenerate the issue? (screenshot/video would be great!)

@faisal-alvi faisal-alvi added the needs:feedback This requires feedback to determine next steps. label May 27, 2022
@foliovision
Copy link
Author

Hello @faisal-alvi

here's the screenshot of the bbPress profile editing page:

bbpress-profile-editing

The URL is like https://site.com/forums/users/{user-name}/edit

Thanks,
Martin

@github-actions github-actions bot removed the needs:feedback This requires feedback to determine next steps. label May 30, 2022
@faisal-alvi
Copy link
Member

Closing as covered in #134

@faisal-alvi faisal-alvi closed this Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants