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

fix: only remove token from hash after login with social media #499

Merged
merged 18 commits into from
Apr 19, 2024

Conversation

mirovladimitrovski
Copy link
Collaborator

@mirovladimitrovski mirovladimitrovski commented Apr 16, 2024

Description

This PR solves OWA-74.

Steps completed:

According to our definition of done, I have completed the following steps:

  • Acceptance criteria met
  • Unit tests added
  • Docs updated (including config and env variables)
  • Translations added
  • UX tested
  • Browsers / platforms tested
  • Rebased & ready to merge without conflicts
  • Reviewed own code

@mirovladimitrovski mirovladimitrovski changed the title fix: only remove token from cache after social media login OWA-74: fix: only remove token from cache after login with social media Apr 16, 2024
Copy link

github-actions bot commented Apr 16, 2024

Visit the preview URL for this PR (updated for commit 6d3c449):

https://ottwebapp--pr499-owa-74-only-remove-t-o7m3qo9c.web.app

(expires Sat, 18 May 2024 19:18:02 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c198f8a3a199ba8747819f7f1e45cf602b777529

@mirovladimitrovski mirovladimitrovski changed the title OWA-74: fix: only remove token from cache after login with social media OWA-74: fix: only remove token from hash after login with social media Apr 16, 2024
@ChristiaanScheermeijer
Copy link
Collaborator

Don't forget to squash and give the commit(s) a nice name for the changelogs 😄 I think the changelog parser skips the OWA-<num>: message commits currently in the develop branch.

@AntonLantukh AntonLantukh changed the title OWA-74: fix: only remove token from hash after login with social media fix: only remove token from hash after login with social media Apr 18, 2024
@mirovladimitrovski mirovladimitrovski requested review from AntonLantukh and removed request for AntonLantukh April 18, 2024 10:01
Copy link
Collaborator

@AntonLantukh AntonLantukh left a comment

Choose a reason for hiding this comment

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

Nice changes, thank you!

@AntonLantukh AntonLantukh self-requested a review April 19, 2024 09:19
@mirovladimitrovski mirovladimitrovski merged commit 7438bc7 into develop Apr 19, 2024
10 checks passed
@mirovladimitrovski mirovladimitrovski deleted the OWA-74-only-remove-token-from-hash branch April 19, 2024 09:22
AntonLantukh pushed a commit that referenced this pull request Apr 22, 2024
This was referenced Apr 22, 2024
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