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 Org Image Upload #1216

Merged
merged 9 commits into from
Dec 30, 2023
Merged

Fix Org Image Upload #1216

merged 9 commits into from
Dec 30, 2023

Conversation

elraphty
Copy link
Contributor

Describe your changes

Moved the token refresh to App.tsx to enable token refresh in child components

Issue ticket number and link

Closes #1068

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested on Chrome and Firefox
  • I have tested on a mobile device

@kevkevinpal
Copy link
Contributor

you might want to remove frontend/app/coverage from the diff

we should remove it from the project in general to be honest

@elraphty
Copy link
Contributor Author

you might want to remove frontend/app/coverage from the diff

we should remove it from the project in general to be honest

Oh! thanks, I was wondering what must have changed.

@elraphty elraphty merged commit c806d24 into master Dec 30, 2023
5 checks passed
@ecurrencyhodler
Copy link
Contributor

I still can't update photo in sphinx app org even though i have the right perms.

@elraphty
Copy link
Contributor Author

I still can't update the photo in sphinx app org even though I have the right perms.
Just tried I can't seem to click on the edit button for Sphinx App ORg, can you give me the right permissions so I can try uploading from my end?

@ecurrencyhodler
Copy link
Contributor

@elraphty I'm going to add you to a different org. try on "sphinx demo org".

@elraphty
Copy link
Contributor Author

@elraphty I'm going to add you to a different org. try on "sphinx demo org".

It's working now.

elraphty added a commit that referenced this pull request Jan 26, 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.

Unable to upload a picture to an organization even with right permissions
3 participants