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

Request refresh tokens #7984

Draft
wants to merge 17 commits into
base: dev-mail
Choose a base branch
from
Draft

Request refresh tokens #7984

wants to merge 17 commits into from

Conversation

paw-hub
Copy link
Contributor

@paw-hub paw-hub commented Nov 20, 2024

Closes #7953

BijinDev and others added 15 commits November 18, 2024 13:24
Issue caused by inline images not being replaced after a loadAll call
(without notify) which sanitizes mail body.
This is due to the replaceInlineImages call being inside a
loadCompleteNotification listener.

Fixed by calling loadAll *with* notify in reply. loadAll is also now only
called when necessary.

Close #6523

Co-authored-by: ivk <[email protected]>
Close #7692
Close #6568

Co-authored-by: bir <[email protected]>
Co-authored-by: hrb-hub <[email protected]>
Crash caused by an uncaught exception due to popover requiring position
information on iPad only but not on iPhone.

Close #7914

Co-authored-by: paw-hub <[email protected]>
On some systems, a backend may not be detected correctly. In a lot of
cases, this can occur if the user is not using one of a hardcoded list
of desktop environments (you can see this list in xdg_util.cc as well
as key_storage_util_linux.cc in GetDesktopEnvironment and SelectBackend
respectively in Chromium's source code), and it determines this using
environment variables.

Basically, if Chromium is unable to detect your desktop environment this
way, safeStorage will not be available in any way except basic_text.

We don't really have any control over this outside of the
preselectGnomeLibsecret() method which may or may not work, but users
can override it with CLI args or just set an app password.

As such, this commit allows basic_text to be used as a backend.

Closes #7257
Issue caused by caching the initially selected language and not changing
current language while switching language. Resolved by resetting
lazyloaded when switching the system language.

close: 7335
Made congratulation.svg same size as other svg files in welcome page to
avoid different image sizes

close: #6853

Co-authored-by: ivk <[email protected]>
This was passing in getNextHalfHour() which takes the current date, not
the date from the viewmodel.

close: #7962

Co-authored-by: paw <[email protected]>
"Calling this method has no effect." according to Apple docs. This was
changed recently according to wayback machine.

https://developer.apple.com/documentation/uikit/uiapplication/1622961-openurl

Fixes #7944

Co-authored-by: paw-hub <[email protected]>
The inner dropdown container was overflowing the outer one instead of
scrolling because its height was not restricted. This could be
reproduced on a screen that's not tall enough e.g. small mobile
screen or landscape orientation of the phone.

Now we set the height to the one of the parent to always have correct
height.

Close #7937
@paw-hub paw-hub linked an issue Nov 20, 2024 that may be closed by this pull request
while (true) {
// Don't want to use entityClient.loadAll in case the token expires in the middle of us getting entities and
// then we'd have to do it all over again...
const elements = await this.handleRequest(mailGroup, async (options) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

we need to load and store them chunk by chunk so loadAll doesn't make sense anyway

Copy link
Contributor Author

Choose a reason for hiding this comment

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

noted

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.

Requesting & refreshing export tokens
4 participants