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: [IOCOM-980] Add Timeline component #5426

Merged
merged 6 commits into from
Jan 26, 2024
Merged

Conversation

adelloste
Copy link
Member

@adelloste adelloste commented Jan 23, 2024

Short description

This PR adds Timeline, a component that is going to be used in the detail of a message.

Details

timeline

List of changes proposed in this pull request

  • Added Timeline

How to test

Tests should be solved correctly

@adelloste adelloste requested a review from a team as a code owner January 23, 2024 14:12
@pagopa-github-bot pagopa-github-bot changed the title [IOCOM-980] Add Timeline component feat: [IOCOM-980] Add Timeline component Jan 23, 2024
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Jan 23, 2024

Affected stories

  • 🌟 IOCOM-980: [App] Creare il componente timeline
    subtask of
    • IOCOM-814: Adottare il nuovo design system nel dettaglio di un messaggio di SEND

Generated by 🚫 dangerJS against 1ad7868

Copy link

codecov bot commented Jan 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (973f8e7) 47.32% compared to head (ee47299) 47.35%.
Report is 1 commits behind head on master.

❗ Current head ee47299 differs from pull request most recent head 1ad7868. Consider uploading reports for the commit 1ad7868 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5426      +/-   ##
==========================================
+ Coverage   47.32%   47.35%   +0.03%     
==========================================
  Files        1509     1510       +1     
  Lines       32055    32074      +19     
  Branches     7842     7845       +3     
==========================================
+ Hits        15171    15190      +19     
  Misses      16826    16826              
  Partials       58       58              
Files Coverage Δ
ts/features/pn/components/Timeline.tsx 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 942e98b...1ad7868. Read the comment docs.

Copy link
Contributor

@Vangaorth Vangaorth left a comment

Choose a reason for hiding this comment

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

LGTM

@adelloste adelloste merged commit 4b708e4 into master Jan 26, 2024
6 checks passed
@adelloste adelloste deleted the IOCOM-980-timeline branch January 26, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants