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

fix(eslint): fix malformed html #3379

Merged
merged 1 commit into from
May 31, 2022
Merged

fix(eslint): fix malformed html #3379

merged 1 commit into from
May 31, 2022

Conversation

josephmcg
Copy link
Contributor

What this PR does πŸ“–

  • fix non self-closing input and br with leading /
  • prettier format files

Which issue(s) this PR fixes πŸ”¨
AP-1661

Special notes for reviewers πŸ—’οΈ

Additional comments 🎀

@github-actions github-actions bot added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label May 31, 2022
@netlify
Copy link

netlify bot commented May 31, 2022

βœ… Yeeeehaw, deploy preview is ready!

Name Link
πŸ”¨ Latest commit 516c7f4
πŸ” Latest deploy log https://app.netlify.com/sites/adoring-edison-dbcef8/deploys/62958c7981fd140008be34f8
😎 Deploy Preview https://deploy-preview-3379--adoring-edison-dbcef8.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@josephmcg josephmcg changed the title fix(eslint): clean malformed html fix(eslint): fix malformed html May 31, 2022
<!-- <InteractablesInput
<div class="input-container">
<span class="label">{{$t('servers.create.select_friends')}}</span>
<!-- <InteractablesInput
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we delete the commented code here?

@Jekrimo Jekrimo removed the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label May 31, 2022
@stavares843 stavares843 added the Ready for QA Ready for QA team to test, Devs approved. label May 31, 2022
@phillsatellite phillsatellite added QA tested One QA Person has tested - Needs QA Lead review still and removed Ready for QA Ready for QA team to test, Devs approved. labels May 31, 2022
@stavares843 stavares843 merged commit ee82dff into dev May 31, 2022
@stavares843 stavares843 deleted the AP1661 branch May 31, 2022 16:16
@github-actions github-actions bot removed the QA tested One QA Person has tested - Needs QA Lead review still label May 31, 2022
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

4 participants