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

feat: perform "back" actions througout the app using centralized hook (hl-1276) #3394

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sirtawast
Copy link
Collaborator

@sirtawast sirtawast commented Oct 3, 2024

Description ✨

Handler side changes. Back buttons each had custom function calls to next router. Use centralized hook to perform these actions with different logic and use cases.

  • Use one function to invoke when Back buttons are pressed
    • Usually, go back from application to corresponding tab with matching status
    • Go to archive if the application is archived
      • One special case where similar applications were searched for with ?appNo=123456 param
    • Go to alterations if it was previously visited
    • If no other means is provided, route back to root of the app
  • Browsing is now recorded to window.sessionStorage to allow for history tracking and usage where needed
  • Fix one key prop issue

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://kesaseteli-pr3394.api.dev.hel.ninja/healthz 🚀🚀🚀

Copy link

sonarcloud bot commented Oct 4, 2024

@terovirtanen
Copy link
Contributor

YJDH-TET-API branch is deployed to platta: https://yjdh-tet-pr3394.api.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

ADMIN is deployed to platta: https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

YOUTH branch is deployed to platta: https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://helsinkilisa-pr3394.api.dev.hel.ninja/healthz 🚀🚀🚀

@terovirtanen
Copy link
Contributor

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://helsinkilisa-ui-pr3394.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://helsinkilisa-ui-handler-pr3394.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

YJDH-TET-API branch is deployed to platta: https://yjdh-tet-pr3394.api.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

YOUTH branch is deployed to platta: https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

ADMIN is deployed to platta: https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

TestCafe result is success for https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://helsinkilisa-pr3394.api.dev.hel.ninja/healthz 🚀🚀🚀

@terovirtanen
Copy link
Contributor

YJDH-TET-API branch is deployed to platta: https://yjdh-tet-pr3394.api.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

YOUTH branch is deployed to platta: https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

ADMIN is deployed to platta: https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe youth result is failed for https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

TestCafe admin result is failed for https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 😿💢💥💥

Copy link

sonarcloud bot commented Oct 21, 2024

Quality Gate Failed Quality Gate failed for 'yjdh'

Failed conditions
8.8% Coverage on New Code (required ≥ 65%)

See analysis details on SonarCloud

Copy link

sonarcloud bot commented Oct 21, 2024

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://kesaseteli-pr3394.api.dev.hel.ninja/healthz 🚀🚀🚀

Copy link

sonarcloud bot commented Oct 21, 2024

Copy link

sonarcloud bot commented Oct 21, 2024

Quality Gate Failed Quality Gate failed for 'yjdh'

Failed conditions
33.3% Coverage on New Code (required ≥ 65%)

See analysis details on SonarCloud

Copy link

sonarcloud bot commented Oct 21, 2024

Copy link

sonarcloud bot commented Oct 21, 2024

Copy link

sonarcloud bot commented Oct 21, 2024

Copy link

sonarcloud bot commented Oct 21, 2024

@terovirtanen
Copy link
Contributor

YJDH-TET-API branch is deployed to platta: https://yjdh-tet-pr3394.api.dev.hel.ninja 🚀🚀🚀

Copy link

sonarcloud bot commented Oct 21, 2024

Copy link

sonarcloud bot commented Oct 21, 2024

Copy link

sonarcloud bot commented Oct 21, 2024

@terovirtanen
Copy link
Contributor

YOUTH branch is deployed to platta: https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

ADMIN is deployed to platta: https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe youth result is failed for https://yjdh-tet-youth-ui-pr3394.dev.hel.ninja 😿💢💥💥

Copy link

sonarcloud bot commented Oct 21, 2024

@terovirtanen
Copy link
Contributor

TestCafe admin result is failed for https://yjdh-tet-admin-ui-pr3394.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

EMPLOYER branch is deployed to platta: https://kesaseteli-employer-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe employer result is failed for https://kesaseteli-employer-pr3394.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3394.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

API branch is deployed to platta: https://helsinkilisa-pr3394.api.dev.hel.ninja/healthz 🚀🚀🚀

@terovirtanen
Copy link
Contributor

TestCafe applicant result is failed for https://helsinkilisa-ui-pr3394.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

TestCafe handler result is failed for https://helsinkilisa-ui-handler-pr3394.dev.hel.ninja 😿💢💥💥

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.

3 participants