Commit graph

9795 commits

Author SHA1 Message Date
Manav Rathi e8dfbdd581 Fix source maps 2024-02-10 21:34:24 +05:30
Manav Rathi d54ff83bba Resurrect SENTRY_AUTH_TOKEN 2024-02-10 19:47:59 +05:30
Manav Rathi c34654e995 Further prune down Sentry 2024-02-10 17:38:33 +05:30
Manav Rathi 0119cddd50 Prune 2024-02-10 17:26:23 +05:30
Manav Rathi 977da14e70 Namespace more 2024-02-10 16:19:06 +05:30
Manav Rathi 36e76626cd Unconditionally use environment overrides 2024-02-10 16:11:19 +05:30
Manav Rathi 1dac60aeb2 Namespace vars 2024-02-10 15:51:05 +05:30
Manav Rathi fc695eff75 Rename Deployment to Build
Perhaps (perhaps) this better conveys the fact that the build itself is a static
entity that doesn't and cannot change after deployment.
2024-02-10 15:33:10 +05:30
Manav Rathi 3ad29a504c Move to a singular flag
This might not be enough if we also want to cater for development desktop
builds, but let's start with this and add them back later if needed.
2024-02-10 15:32:15 +05:30
Manav Rathi c921399540 WIP 2024-02-10 15:21:06 +05:30
Manav Rathi badece4cce Understand a bit better 2024-02-10 10:36:58 +05:30
Manav Rathi 3b3804b655
Set the correct Sentry tag when deploying auth-release (#1609) 2024-02-09 17:55:52 +05:30
Manav Rathi 9acc0756c9 Set the correct Sentry tag when deploying auth-release
...in the photos-web CF pages project.
2024-02-09 17:54:26 +05:30
Manav Rathi e59e7436eb
Reduce Sentry noise (#1606) 2024-02-09 16:37:08 +05:30
Manav Rathi 4c469c853a Update to latest DSNs 2024-02-09 16:33:23 +05:30
Manav Rathi 7a0422151c Remove unused env var 2024-02-09 15:19:11 +05:30
Manav Rathi 88b4aa02c7 Inline constant 2024-02-09 13:33:18 +05:30
Manav Rathi 9289bbd0d0 Reduce Sentry DSN noise
We don't need to customize this, so reduce all the indirection and dance around it.
2024-02-09 13:32:28 +05:30
Manav Rathi e5f8b5faa8 Remove unused sentry ENV var 2024-02-09 11:44:42 +05:30
Manav Rathi fa93b3ffee
Update the UI when deleting a subset of the duplicates (#1605) 2024-02-09 11:14:22 +05:30
Manav Rathi 3ef34a4e97 Update the UI when deleting a subset of the duplicates
The list of photos shown on the dedup screen did not update if we selected a
subset of the duplicates and deleted them. This is because while we're issuing a
remote API request, we don't seem to be updating our local storage with the
results of that when we're on this screen. So when the gallery on this screen
re-renders, it still reads the same files again, even after they've been
deleted.

Not happy with the current solution, but it does solve the customers issue. Have
added an "Enhancement" tag to flag this.

Test:

I was able to reproduce the issue. After making these changes, I don't see it
anymore. Haven't done exhaustive checks, but it fixes the happy path at least.

Tested locally.
2024-02-09 11:08:14 +05:30
httpjamesm a5329ca410
fix: don’t require encrypted token 2024-02-08 16:11:44 -05:00
httpjamesm 9852e3cf37
Merge branch 'main' 2024-02-08 16:04:28 -05:00
httpjamesm d02a0611c7
fix: make user const 2024-02-08 15:59:56 -05:00
httpjamesm 5b1eb3df9b
feat: get and pass accounts token 2024-02-08 15:56:40 -05:00
httpjamesm 688c840af6
feat: get accounts token and pass to passkeys url 2024-02-08 15:46:33 -05:00
httpjamesm ad1fab8edb
feat: getAccountsURL() 2024-02-08 15:42:51 -05:00
Manav Rathi e0ecbba2e7
Remove unused code and files (#1604) 2024-02-08 15:41:45 +05:30
Manav Rathi 878b7a5e32 Remove unused PWA / manifest code
Had discussed this with Abhinav, there were some issues around marking our app
as a PWA, tied to worker lifetime, so the PWA functionality had been disabled,
and these files are not currently not supposed to be used.
2024-02-08 14:48:43 +05:30
Manav Rathi b4946693f8 Remove unused workbox-* dependencies
yarn workspace photos remove workbox-precaching workbox-recipes workbox-routing workbox-strategies workbox-window
2024-02-08 14:46:26 +05:30
Manav Rathi f8f35309d0 Remove unused deepsource.toml
This was used a long back for code analysis, but has been disabled for a while,
and the integration with the Deepsource app in the web's GitHub repository has
also been removed.
2024-02-08 14:31:01 +05:30
Manav Rathi 7225071b64 Remove unused LS_KEYS.API_ENDPOINT
This is currently unused. We can resurrect this at a later point when we could
want to make the API we connect to dynamically changable within the app settings
itself, but right now that won't be trivial (e.g. mixing credentials across
instances). So just removing this.
2024-02-08 14:24:57 +05:30
Manav Rathi 67d058d77c
Document env vars (#1601) 2024-02-08 10:13:14 +05:30
Abhinav Kumar a88c7212a0
Fix the zip file related tests (#1602) 2024-02-08 10:08:30 +05:30
Manav Rathi d928429276 Add comments around the purpose of the test 2024-02-07 19:30:47 +05:30
Manav Rathi 5c93c14a02 Fix the zip file related tests
These functions (e.g. getElectronFilesFromGoogleZip) look to have been moved
from `importService` to `ElectronAPIs` in commit
86d61060b7, perhaps the test did not get updated.

The test runs now from the desktop app.
2024-02-07 17:49:33 +05:30
Manav Rathi cdb1fb4f82
New Translations (#1600) 2024-02-07 16:27:06 +05:30
Manav Rathi fb0144466f Add more details 2024-02-07 12:51:56 +05:30
Manav Rathi 6a1c82429a Remove unused auth-url 2024-02-07 12:51:56 +05:30
Manav Rathi 9e71916edf Add more comments and variables 2024-02-07 12:51:56 +05:30
Manav Rathi 009652ac76 Create a sample .env.development 2024-02-07 12:51:56 +05:30
Crowdin Bot e404b12b34 New Crowdin translations by GitHub Action 2024-02-07 00:14:25 +00:00
Manav Rathi 996b65338d
Deploy auth from the same repository (#1599) 2024-02-03 17:25:05 +05:30
Manav Rathi a2c9cd5071 Use the correct casing
-r also worked, but -R is the preferred casing

https://pubs.opengroup.org/onlinepubs/9699919799/utilities/cp.html
2024-02-03 17:12:25 +05:30
Manav Rathi e083c7caf2 Copy the out directory
The symlink does not seem to be working. In the build logs there is nothing
wrong, but at the end CF gives up by saying:

    + ln -sf apps/auth/out
    Finished
    Note: No functions dir at /functions found. Skipping.
    Validating asset output directory
    Failed: build output directory contains links to files that can't be accessed
2024-02-03 16:49:50 +05:30
Manav Rathi 7bd332bad2 POSIX test requires a single =
https://pubs.opengroup.org/onlinepubs/009695399/utilities/test.html

Fixes the following error when running on CF pages:

    ./scripts/deploy.sh: 14: test: multi-deploy-test: unexpected operator

Also xtrace the commands for the log
2024-02-03 16:35:33 +05:30
Manav Rathi d3cf5a9afb Make the deploy script executable 2024-02-03 16:14:23 +05:30
Manav Rathi 67624839f3 Handle the single output directory by creating a symlink 2024-02-03 15:54:45 +05:30
Manav Rathi e858ae2dc6 Deploy auth from the same repository 2024-02-03 11:34:34 +05:30
httpjamesm 97ab0e3be9
fix: add try catch to context.start() 2024-02-02 22:17:03 -05:00