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.

Sign up for GitHub

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

Jump to bottom

Bug fixes - User Setting / View Person / CVS Import #6581

Merged
merged 8 commits into from
Oct 28, 2023
Merged

Bug fixes - User Setting / View Person / CVS Import #6581

merged 8 commits into from
Oct 28, 2023

Conversation

DawoudIO
Copy link
Contributor

@DawoudIO DawoudIO commented Oct 28, 2023 •
edited
Loading

What does PR do? Any background context you want to provide?

  • Removed Clear Persons and Families from CVS Import.
  • View Person - Buttons no longer have a background
  • removed reference to Tooltips.js as it does not exist
  • User Setting cleanup
    • fixed user setting page look and feel
    • system local now displays the local Name
    • removed style selection for user

Screenshots (if appropriate)

image
image
image

@DawoudIO DawoudIO added the bug label Oct 28, 2023
@DawoudIO DawoudIO added this to the 5.0.4 milestone Oct 28, 2023
@DawoudIO DawoudIO self-assigned this Oct 28, 2023
@DawoudIO DawoudIO added the UI label Oct 28, 2023
@DawoudIO DawoudIO changed the title 5.0.3 Ui Bug fixes Bug fixes - User Setting / View Person / CVS Import Oct 28, 2023
@DawoudIO DawoudIO merged commit f63f91c into master Oct 28, 2023
4 checks passed
@DawoudIO DawoudIO deleted the ui/cleanup-oct-28 branch October 28, 2023 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crossan007 crossan007 Awaiting requested review from crossan007

@MrClever MrClever Awaiting requested review from MrClever

@DAcodedBEAT DAcodedBEAT Awaiting requested review from DAcodedBEAT

Assignees

@DawoudIO DawoudIO

Labels
bug UI
Projects
None yet
Milestone
5.0.4
Development

Successfully merging this pull request may close these issues.
1 participant
@DawoudIO