Skip to content

fix(ui/message): only show image files#413

Merged
RhysSullivan merged 18 commits into
AnswerOverflow:mainfrom
JoltCode:only-show-image-files
May 31, 2023
Merged

fix(ui/message): only show image files#413
RhysSullivan merged 18 commits into
AnswerOverflow:mainfrom
JoltCode:only-show-image-files

Conversation

@JoltCode
Copy link
Copy Markdown
Collaborator

@JoltCode JoltCode commented May 27, 2023 •
edited
Loading

Description

  • Only shows files which are:
    • png
    • jpg
    • gif
    • bmp
    • jpeg

Fixes N/A (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

N/A

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated .env.example if I added a new environment variable
  • My PR title follows the semantic commits style

JoltCode added 14 commits May 7, 2023 18:29
@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
answer-overflow-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 31, 2023 11:30am

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2023

@JoltCode is attempting to deploy a commit to the Answer Overflow Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added ⚛ component Issue relates to a react component 🖥️website Relates to the website labels May 27, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2023 •
edited
Loading

Codecov Report

Merging #413 (e5dfbfc) into main (0aa3c16) will increase coverage by 70.10%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           main     #413       +/-   ##
=========================================
+ Coverage      0   70.10%   +70.10%     
=========================================
  Files         0       73       +73     
  Lines         0     2405     +2405     
  Branches      0      609      +609     
=========================================
+ Hits          0     1686     +1686     
- Misses        0      646      +646     
- Partials      0       73       +73

see 73 files with indirect coverage changes

@RhysSullivan RhysSullivan merged commit 7ecbfd8 into AnswerOverflow:main May 31, 2023
RhysSullivan pushed a commit that referenced this pull request Dec 20, 2025
@JoltCode
fix(ui/message): only show image files (#413)
9157468
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

⚛ component Issue relates to a react component 🖥️website Relates to the website

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JoltCode @RhysSullivan