Skip to content

Added preview step for new share account workflow.#2511

Closed
arushi2702 wants to merge 1 commit intoopenMF:developmentfrom
arushi2702:MIFOSAC-546
Closed

Added preview step for new share account workflow.#2511
arushi2702 wants to merge 1 commit intoopenMF:developmentfrom
arushi2702:MIFOSAC-546

Conversation

@arushi2702
Copy link

Fixes - Jira-#MIFOSAC-546

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the static analysis check ./gradlew check or ci-prepush.sh to make sure you didn't break anything
    (Could not run locally due to environment setup issues)
  • If you have multiple commits please combine them into one commit by squashing them.

@biplab1
Copy link
Contributor

biplab1 commented Oct 3, 2025

@arushi2702 Please include screenshots or a screen recording. Also, run ./ci-prepush.sh and commit any resulting changes before pushing the PR to avoid any Github Actions checks failure.

Copy link
Contributor

@revanthkumarJ revanthkumarJ left a comment

Choose a reason for hiding this comment

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

I am not sure because i don't know xml, but i think you wrote in xml in android Main.

But this project is Kotlin multi Platform using jetpack compose. and the ui code should be placed inside commonMain .

@Arinyadav1
Copy link
Contributor

@arushi2702 are you working on these ?

@amanna13
Copy link
Contributor

@therajanmaurya This PR can be closed with reference to #2549. This has already been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants