Failed to Parsingという文法のミスを含んだエラーメッセージを一掃した #4076
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@ot0m1 |
Aseiide
force-pushed
the
feature/fix-console-warn-error-message
branch
from
6433871 to
eab15ff
Compare
ot0m1
reviewed
Feb 6, 2022
There was a problem hiding this comment.
@Aseiide レビューが遅れて大変すいません🙇♂️
app/javascript/admin_companies.vueapp/javascript/company-users.vueapp/javascript/generation-users.vue
の3ファイルにconsole.warn('Failed to parsing', error)という表記が残っているようですが、修正不要でしょうか。
また、
app/jabascript/announcement.vue の 93行目をthis.announcements.push(announcement)からthis.announcements(announcement) に一部改変する
上記のパスに誤りがあるようです。app/javascript/announcements.vueのように思われます。
|
@ot0m1 レビューありがとうございます。
こちら、自分の修正漏れですね。修正します。
typoしてますね。Descriptionの方は既に修正しました。 |
|
@ot0m1 修正もれがあったのを再度修正しました。レビューお願いいたします。 |
ot0m1
approved these changes
Feb 6, 2022
|
@komagata |
54 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Failed to Parsingという文法のミスを含んだエラーメッセージを一掃したい #3920変更前
chrome devtoolのコンソールのスクショ
変更後
chrome devtoolのコンソールのスクショ
上記を確認するための再現方法
ファイルを一部改変してわざとエラーを起こすことで確認することができます
app/javascript/announcements.vueの93行目をthis.announcements.push(announcement)からthis.announcements(announcement)に一部改変するannouncement.vueでなくても任意のファイルを一部改変で大丈夫ですやったこと
console.warn(error)となるように変更しました。もともとはエラーメッセージを
Failed to Parseに書き換える、というissueだったのですがこちらのコメントのように、Failed to Parseすらも不要ということでしたので、console.warn(error)というふうに書き換えを行いました。