Skip to content

fix: encode hash on getUserPhoto#3408

Merged
Mnickii merged 3 commits intomainfrom
fix/email-hash
Mar 10, 2025
Merged

fix: encode hash on getUserPhoto#3408
Mnickii merged 3 commits intomainfrom
fix/email-hash

Conversation

@Mnickii
Copy link
Copy Markdown
Collaborator

@Mnickii Mnickii commented Feb 28, 2025

Closes #3405

PR Type

Bugfix

Description of the changes

PR checklist

  • Project builds (yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)
  • All public APIs (classes, methods, etc) have been documented following the jsdoc syntax
  • Stories have been added and existing stories have been tested
  • Added appropriate documentation. Docs PR:
  • License header has been added to all new source files (yarn setLicense)
  • Contains NO breaking changes

Other information

@Mnickii Mnickii requested a review from a team as a code owner February 28, 2025 12:04
ElinorW
ElinorW previously approved these changes Feb 28, 2025
@github-project-automation github-project-automation Bot moved this to Done ✔️ in Graph Toolkit Feb 28, 2025
Comment thread packages/mgt-components/src/graph/graph.photos.ts Outdated
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 3, 2025

@Mnickii Mnickii merged commit 9bb59d1 into main Mar 10, 2025
@Mnickii Mnickii deleted the fix/email-hash branch March 10, 2025 08:48
@github-project-automation github-project-automation Bot moved this from Done ✔️ to Shipped 🚀 in Graph Toolkit Mar 10, 2025
@pwmather
Copy link
Copy Markdown

Do you know when 4.5.0 is likely to be released @Mnickii ?

@Mnickii
Copy link
Copy Markdown
Collaborator Author

Mnickii commented Mar 19, 2025

@pwmather yes, v4.5.0 will be available at the end of the next week.

@Mnickii
Copy link
Copy Markdown
Collaborator Author

Mnickii commented Mar 25, 2025

@pwmather the release is now available.

@pwmather
Copy link
Copy Markdown

Thank you @Mnickii

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[BUG] Hash urls for external guest accounts are stripped in requests

4 participants