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

feat(list): context menu #372

Merged
merged 4 commits into from
Mar 5, 2024
Merged

feat(list): context menu #372

merged 4 commits into from
Mar 5, 2024

Conversation

Birkbjo
Copy link
Member

@Birkbjo Birkbjo commented Feb 20, 2024

Copy link

netlify bot commented Feb 20, 2024

Deploy Preview for dhis2-maintenance-app-beta ready!

Name Link
🔨 Latest commit c19814f
🔍 Latest deploy log https://app.netlify.com/sites/dhis2-maintenance-app-beta/deploys/65e768e4366eb8000877b732
😎 Deploy Preview https://deploy-preview-372--dhis2-maintenance-app-beta.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 configuration.

}

/* Wrapping button with anchor-tags are not valid, style anchor as a UI-button */
export const LinkButton = ({
Copy link
Member Author

Choose a reason for hiding this comment

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

This should be done in UI, eg passing an as={'a'} or something similar to button component...

@Birkbjo Birkbjo requested a review from Mohammer5 March 1, 2024 18:39
@Birkbjo Birkbjo force-pushed the feat/list-context-menu branch from 77460bd to c19814f Compare March 5, 2024 18:48
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 58.97436% with 16 lines in your changes are missing coverage. Please review.

Project coverage is 72.95%. Comparing base (849de93) to head (c19814f).

Files Patch % Lines
...nts/sectionList/listActions/SectionListActions.tsx 57.14% 9 Missing ⚠️
src/components/LinkButton/LinkButton.tsx 64.28% 5 Missing ⚠️
src/components/sectionList/SectionListWrapper.tsx 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #372      +/-   ##
==========================================
- Coverage   73.30%   72.95%   -0.36%     
==========================================
  Files         141      143       +2     
  Lines        1967     2000      +33     
  Branches      405      411       +6     
==========================================
+ Hits         1442     1459      +17     
- Misses        523      539      +16     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Birkbjo Birkbjo merged commit f5aa068 into master Mar 5, 2024
9 checks passed
@Birkbjo Birkbjo deleted the feat/list-context-menu branch March 5, 2024 18:54
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants