Skip to content

Fix current user bug #3321

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

Merged
merged 2 commits into from
Apr 29, 2025
Merged

Fix current user bug #3321

merged 2 commits into from
Apr 29, 2025

Conversation

cammiida
Copy link
Contributor

@cammiida cammiida commented Apr 25, 2025

Description

Was using useCurrentParty even though I've been the one that has dealt with that and seen that it should not be used inside of an instance. Was causing some bugs, but they should be fixed now after using useProfile instead.

Related Issue(s)

  • closes #{issue number}

Verification/QA

  • Manual functionality testing
    • I have tested these changes manually
    • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
    • No testing done/necessary
  • Automated tests
    • Unit test(s) have been added/updated
    • Cypress E2E test(s) have been added/updated
    • No automatic tests are needed here (no functional changes/additions)
    • I want someone to help me make some tests
  • UU/WCAG (follow these guidelines until we have our own)
    • I have tested with a screen reader/keyboard navigation/automated wcag validator
    • No testing done/necessary (no DOM/visual changes)
    • I want someone to help me perform accessibility testing
  • User documentation @ altinn-studio-docs
    • Has been added/updated
    • No functionality has been changed/added, so no documentation is needed
    • I will do that later/have created an issue
  • Support in Altinn Studio
    • Issue(s) created for support in Studio
    • This change/feature does not require any changes to Altinn Studio
  • Sprint board
    • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
    • I don't have permissions to do that, please help me out
  • Labels
    • I have added a kind/* and backport* label to this PR for proper release notes grouping
    • I don't have permissions to add labels, please help me out

@cammiida cammiida added kind/bug Something isn't working backport-ignore This PR is a new feature and should not be cherry-picked onto release branches labels Apr 25, 2025
@cammiida cammiida moved this to 🔎 Review in Team Apps Apr 25, 2025
@cammiida cammiida force-pushed the signing/rename-signingstatuspanel branch from 6a4a399 to febaf6a Compare April 25, 2025 12:31
@cammiida cammiida force-pushed the signing/fix-current-user-bug branch from 352d7f3 to 2117867 Compare April 25, 2025 15:20
Base automatically changed from signing/rename-signingstatuspanel to feature/signing April 25, 2025 15:44
@cammiida cammiida force-pushed the signing/fix-current-user-bug branch from 2117867 to 9c4f6b2 Compare April 25, 2025 15:47
@cammiida cammiida self-assigned this Apr 28, 2025
@cammiida cammiida merged commit 483d576 into feature/signing Apr 29, 2025
15 checks passed
@cammiida cammiida deleted the signing/fix-current-user-bug branch April 29, 2025 09:31
@github-project-automation github-project-automation bot moved this from 🔎 Review to 🧪 Test in Team Apps Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-ignore This PR is a new feature and should not be cherry-picked onto release branches kind/bug Something isn't working
Projects
Status: 🧪 Test
Development

Successfully merging this pull request may close these issues.

2 participants