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

A11y #1580

Merged
merged 5 commits into from
May 20, 2021
Merged

A11y #1580

merged 5 commits into from
May 20, 2021

Conversation

fritzSF
Copy link
Contributor

@fritzSF fritzSF commented May 18, 2021

No description provided.

@fritzSF fritzSF requested a review from MarkLark86 May 18, 2021 10:35
client/components/Main/SearchPanel.tsx Outdated Show resolved Hide resolved
client/components/NotificationModal.tsx Outdated Show resolved Hide resolved
client/components/SelectItemModal.tsx Outdated Show resolved Hide resolved
client/apps/PageContent.tsx Outdated Show resolved Hide resolved
@MarkLark86
Copy link
Collaborator

@fritzSF Other PRs are building. The only thing with your PR that may have an affect is the inclusion of the package-lock.json file. Try removing that and re-push

@fritzSF
Copy link
Contributor Author

fritzSF commented May 20, 2021

@MarkLark86 It seams that package-lock.json was indeed the problem, thanks. The build passes now.

@MarkLark86 MarkLark86 added this to the 2.3 milestone May 20, 2021
@pavlovicnemanja pavlovicnemanja merged commit 32d57ac into superdesk:develop May 20, 2021
@MarkLark86 MarkLark86 added the merged Merged into release branch label Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged Merged into release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants