Skip to content

Conversation

@muhammad-ali-pk
Copy link
Contributor

@muhammad-ali-pk muhammad-ali-pk commented Jan 15, 2025

Done

  • Checked the existence of logged in user in "users" table, and if negative, fetched user info from canonical directory and saved in users table
  • Stored user id in session
  • Added an endpoint /api/current-user to fetch currently logged in user's info
  • Added a helper function called get_user_from_directory_by_key which fetches a user from canonical directory by a given key,value.

QA

QA steps

Please note that you need to test it locally, because the directory API on demo is currently not working and is being fixed in a different task.

  • Check out this PR and run the application using dotrun command locally.
  • Open localhost:8104 in your browser.
  • Log in using Ubuntu SSO if not already.
  • Make sure you have the VPN on. (This is required to fetch data from directory API)
  • Call the localhost:8104/api/current-user GET endpoint and verify it returns the currently logged in user's info.

Fixes

@webteam-app
Copy link

@muhammad-ali-pk muhammad-ali-pk force-pushed the WD-18146 branch 2 times, most recently from 144822b to 044a174 Compare January 15, 2025 05:08
@muhammad-ali-pk muhammad-ali-pk merged commit aee6fe0 into main Jan 15, 2025
7 checks passed
@muhammad-ali-pk muhammad-ali-pk deleted the WD-18146 branch January 15, 2025 08:22
@github-actions
Copy link

github-actions bot commented Sep 5, 2025

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants