Skip to content

Fix closing of the banner component #194

Merged
stevekinney merged 1 commit intouser-iconfrom
fix-close-banner
Jan 4, 2022
Merged

Fix closing of the banner component #194
stevekinney merged 1 commit intouser-iconfrom
fix-close-banner

Conversation

@feedmeapples
Copy link
Copy Markdown
Contributor

What was changed

Fixes closing of banner component. Switched to comparing current version instead of recommended

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@feedmeapples feedmeapples changed the title Fix close banner Fix closing of the banner component Jan 4, 2022
@feedmeapples feedmeapples changed the base branch from main to user-icon January 4, 2022 19:23
@stevekinney stevekinney merged commit e326bca into user-icon Jan 4, 2022
@stevekinney stevekinney deleted the fix-close-banner branch January 4, 2022 19:27
stevekinney pushed a commit that referenced this pull request Jan 4, 2022
* Show authenticated user's avatar

* address comments

* Fix closing of the banner component (#194)
rossnelson added a commit that referenced this pull request Apr 3, 2026
- lodash: ^4.17.21 -> ^4.18.1 (CVE-2026-4800, CVE-2026-2950, CVE-2025-13465)
- svelte: 5.53.3 -> 5.55.1 (CVE-2026-27902, CVE-2026-27901)
- @sveltejs/kit: 2.53.0 -> 2.55.0 (DoS via form deserialization)
- storybook: ^8.6.11 -> ^8.6.18 (CVE-2026-27148)
- tar-fs: >=2.1.2 -> ^3.1.2 (CVE-2025-59343)

Resolves Dependabot alerts #233, #232, #159, #204, #203, #194, #193, #207, #192, #127.
rossnelson added a commit that referenced this pull request Apr 8, 2026
#3269)

- lodash: ^4.17.21 -> ^4.18.1 (CVE-2026-4800, CVE-2026-2950, CVE-2025-13465)
- svelte: 5.53.3 -> 5.55.1 (CVE-2026-27902, CVE-2026-27901)
- @sveltejs/kit: 2.53.0 -> 2.55.0 (DoS via form deserialization)
- storybook: ^8.6.11 -> ^8.6.18 (CVE-2026-27148)
- tar-fs: >=2.1.2 -> ^3.1.2 (CVE-2025-59343)

Resolves Dependabot alerts #233, #232, #159, #204, #203, #194, #193, #207, #192, #127.
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.

2 participants