-
Notifications
You must be signed in to change notification settings - Fork 9
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
User store signed url #228
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Swap siteNotification to use a pinia store, and make the z-index bigger. (Fixes #151) (#152) * Bugs/145 ga disclaimer and visual improvements (#153) * 📜 improving copy * 💚 improve spacing in home view * 💚 improve footer bottom position without scroll bars * 🔨 handle empty schedules list * Bugs/154 calendar sync (#155) * 🔨 only json load google token if it exists * 🔨 fix sync button styles * Improving copy (#156) * Rename instances of thundernest to thunderbird. (#163) * Warn the user about the consequences of disconnecting a calendar. (#160) * Warn the user about the consequences of disconnecting a calendar. * ➕ update locales --------- Co-authored-by: Andreas Müller <[email protected]> * Features/105 GA booking request flow (#159) * ➕ use slots for availability attendees * ➕ extend booking flow for availabilities (wip) * 📜 update er documentation * ➕ implement request confirmation mail * ➕ add booking confirmation view * ➕ create loading spinner component * ➕ availability request handling * ➕ add external link to my link settings * ➕ add booking status * ❌ remove console statement * 📜 update documentation * 🔨 make date format constant * ❌ remove unused code * ➕ add check for previously booked slot * Remove carriage return (#167) * ❌ rmove cr, keep lf * ❌ remove cr, keep lf * Add booking request denial information (#168) --------- Co-authored-by: Mel <[email protected]>
* Add the site title to vue config. (Fixes #180) (#182) * Remove sender arguments from mail (#179) * Our smtp provider doesn't support non-verified senders, so let's simplify this and only support service_email for now. * Add a fake mail server to dev docker * Lang snippet updates for English (#183) * ➕ add lang snippet updates for en * ➕ add lang snippet updates for en --------- Co-authored-by: Andreas <[email protected]>
* Fix the database connection with in-memory db. * Don't automatically create a session instance if we're simply importing database. * Don't automatically create an Auth0 instance if we're simply importing auth * Added calendar, and subscriber factory fixtures * Add integration class tests, and moved general, profile, and caldav tests * Refactored tests to no longer rely on each other, as this can lead to undefined behaviour depending on how the test is ran * Added standard fixtures for retrieving client and db * Update readme with new test instructions * Update the minimum version of python (to 3.11) and fixed some deprecated functions.
* 🔨 fix datetime comparison * ➕ add timezone data when creating new subscriber * ❌ remove logging * 🔨 change login endpoint from get to post
* ➕ component element for nav bar items * ➕ provide unified login and logout methods * 🔨 fix missing import * ➕ add profile dropdown * ➕ add ga link to schedule creation area * 🔨 fix vue warning for extraneous non-props attributes * 💚 add active nav item color gradients * 🔨 fix share my link buttons to copy link into clipboard
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the Change
This change moves the user signed url to the frontend user store as well es function to update the stored signed url and replace it. Existing users might need to generate a new URL in the settings to initially fill the store.
I also fixed an issue with the background color of the user avatar in the navbar and took the opportunity to convert it to a separate component.
Benefits
A lot more cleaner code due to central url handling.
Applicable Issues
Closes #198