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

User story6 frontend #87

Merged
merged 3 commits into from
Oct 20, 2024
Merged

User story6 frontend #87

merged 3 commits into from
Oct 20, 2024

Conversation

DachiCharkviani
Copy link

@DachiCharkviani DachiCharkviani commented Oct 20, 2024

Closes #84, Closes #85

Added the button "manage notes" on every post. Once it is clicked the pop up appears where the current notes appear for the post, or none if there aren't any. The section where we can write new notes and save them also appears. We can open this pop up and close it without adding anything.

@DachiCharkviani DachiCharkviani added the enhancement New feature or request label Oct 20, 2024
@DachiCharkviani DachiCharkviani added this to the Sprint 2 milestone Oct 20, 2024
@DachiCharkviani DachiCharkviani self-assigned this Oct 20, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11429360450

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.005%) to 82.611%

Files with Coverage Reduction New Missed Lines %
src/meta/errors.js 1 76.74%
Totals Coverage Status
Change from base Build 11429178227: -0.005%
Covered Lines: 22345
Relevant Lines: 25625

💛 - Coveralls

Copy link

@Seckhen Seckhen left a comment

Choose a reason for hiding this comment

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

Looks good, ready to merge!

Copy link

@NickDevi NickDevi left a comment

Choose a reason for hiding this comment

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

all good

Copy link

@Seckhen Seckhen left a comment

Choose a reason for hiding this comment

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

Looks good ready to merge!

@DachiCharkviani DachiCharkviani merged commit 157d8e6 into f24 Oct 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
4 participants