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

Add payment status to be displayed for instance in messagebox #389

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Magnusrm
Copy link
Contributor

@Magnusrm Magnusrm commented Apr 24, 2024

Description

Add payment as a status to be displayed for an instance currently in the payment process task.

Related Issue(s)

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@Magnusrm Magnusrm changed the title add payment status to be displayed for instance in messagebox Add payment status to be displayed for instance in messagebox Apr 24, 2024
Copy link

sonarcloud bot commented Apr 24, 2024

@Magnusrm Magnusrm marked this pull request as ready for review April 24, 2024 12:35
@Magnusrm Magnusrm self-assigned this Apr 24, 2024
@SandGrainOne
Copy link
Member

I'm not sure if this would work without changes in Altinn 2. Is that something you're investigating?

@RonnyB71 RonnyB71 linked an issue May 6, 2024 that may be closed by this pull request
2 tasks
@acn-sbuad
Copy link
Contributor

acn-sbuad commented May 14, 2024

This will in the least require updates to the logic here.

And logic in the view might need reworking as well.

@SandGrainOne
Copy link
Member

@olebhansen @RonnyB71 You guys have unfinished business for the Payment step related to how it's reflected in the message box.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🔎 Review
Development

Successfully merging this pull request may close these issues.

Handle pay action in MessageBox to display current instance status
3 participants