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

imprv: Delete NotFoundAlert from not found page #5919

Merged
merged 2 commits into from
May 31, 2022

Conversation

Shunm634-source
Copy link
Contributor

Emptyページにおいてページ作成の案内を消す。
https://redmine.weseek.co.jp/issues/96329

@Shunm634-source Shunm634-source self-assigned this May 30, 2022
@Shunm634-source Shunm634-source temporarily deployed to VRT May 30, 2022 03:43 Inactive
@@ -117,9 +116,6 @@ Object.assign(componentMappings, {

'share-link-alert': <ShareLinkAlert />,
'redirected-alert': <RedirectedAlert />,
'not-found-alert': <NotFoundAlert
Copy link
Member

Choose a reason for hiding this comment

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

swig template からも not-found-alert にあたる DOM を消さないといけない

@Shunm634-source Shunm634-source temporarily deployed to VRT May 31, 2022 02:39 Inactive
@reg-suit
Copy link

reg-suit bot commented May 31, 2022

reg-suit detected visual differences.

Check this report, and review them.

⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪
⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫⚫

What do the circles mean? The number of circles represent the number of changed images.
🔴 : Changed items, ⚪ : New items, ⚫ : Deleted items, and 🔵 Passed items

How can I change the check status? If reviewers approve this PR, the reg context status will be green automatically.

@yuki-takei yuki-takei merged commit 9107250 into master May 31, 2022
@yuki-takei yuki-takei deleted the imprv/NotFoundpage-messageDeletion branch May 31, 2022 05:07
@yuki-takei yuki-takei changed the title imprv: Deleted NotFoundAlert page entirely as well as the import in app.jsx imprv: Delete NotFoundAlert from not found page May 31, 2022
@github-actions github-actions bot mentioned this pull request May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants