jsnbuchanan/crowd-funder-for-time-pwa
1
0
Fork 1
forked from trent_larson/crowd-funder-for-time-pwa
Code Issues Pull Requests Projects Releases Wiki Activity

A couple small fixes, plus a merge from master #1

Merged
jsnbuchanan merged 45 commits from trentlarson/crowd-funder-from-jason:feat/vitejs-trent into feat/vitejs 2024-04-07 17:52:43 +00:00
Conversation 2 Commits 45 Files Changed 37 +3264 -4986
trentlarson commented 2024-04-07 15:50:38 +00:00
Contributor
Copy Link

This includes the following tiny fixes:

All the rest of the changes are due to a merge from master from the original repo. (I did this as a merge instead of a rebase, mostly because I've been experimenting.)

I'm open to other approaches, eg. rebasing, or pulling into your repo from master (which requires some permissions to me), or... whatever.

The only remaining task before merging is to deploy this to my test server and put it through the paces (which I'll do today), but I expect no issues since npm run build and npm run serve already worked.

This includes the following tiny fixes: * a [fix for the contact-download](https://gitea.anomalistdesign.com/trentlarson/crowd-funder-from-jason/commit/0fbd1ad51a9b383b9935b62868d0d354b5985bf4) * a [change to avoid an on-screen error on localhost](https://gitea.anomalistdesign.com/trentlarson/crowd-funder-from-jason/commit/2ae165d56fb9decbeb608872e06896e00abc6731) * a [change to avoid a console error in a common case](https://gitea.anomalistdesign.com/trentlarson/crowd-funder-from-jason/commit/78fab735e6bb331f0e3b7554bbc7edcefeda1f39) All the rest of the changes are due to a merge from master from the original repo. (I did this as a merge instead of a rebase, mostly because I've been experimenting.) I'm open to other approaches, eg. rebasing, or pulling into your repo from master (which requires some permissions to me), or... whatever. The only remaining task before merging is to deploy this to my test server and put it through the paces (which I'll do today), but I expect no issues since `npm run build` and `npm run serve` already worked.
trentlarson added 41 commits 2024-04-07 15:50:39 +00:00
Merge pull request 'fix: npm audit fix to resolve vulnerabilities 1 low, 3 moderate, 1 high' (#108) from jsnbuchanan/crowd-funder-for-time-pwa:master into master cc6d0958dc 
Reviewed-on: trent_larson/crowd-funder-for-time-pwa#108
add a camera-switch button 6dc0c2cd58
fix so that project agent & location removals get saved 2b8f9579f1
bump version to 0.3.5 28754bdfb1
add button on photo to switch to mirror mode 40d12b1f9c
show warnings before dismissing prompt, and add to tasks and help e020caaa50
add blurb explaining what data is shared with the world 4ad4cab25e
change icon for detail view (from circle-info to file-lines) be2154f847
Merge pull request 'add button during photo to switch to mirror mode' (#109) from photo-reverse into master 1fc5b0ea2b 
Reviewed-on: trent_larson/crowd-funder-for-time-pwa#109
add onboarding help instructions as separate page 0e2c5af16e
fix check for more camera-device options 35455e6648
bump to version 0.3.6 3a07e31d63
bump version and add -beta b28104af5b
refactor tasks 2dfc8fedaa
Added close icon to gifted prompts dialog 4e227fc07a
Fixed map z-index 0dab475d8b
Optimized grid space for wider screens e028197a2a
Button width changes dce4d3cc72 
For buttons that are next to each other
Map fix #2 55da3d0b1c
Feed filters dialog 4635c1ac48
Filter options reduced for release 7f66addfe3
Removed one more cfd53bc186
Additions to Account View 0b4f2484f7
add logic to send a time for notifications 7692cc2b35
update tasks e4cf79b558
add adjustment to UTC hour for notification time 121181c6a1
adjust the notification-subscription objects to try and send correct info 2ca33bb9eb
adjust more code to the PushSubscriptionJSON 489bb76a60
Names and variables for filter toggles e90a0be6d9
feed filter: save the changed values to the DB, go to map if no location chosen, reload if necessary 8e1daf7015
Merge pull request 'ui-fixes-2024-03' (#111) from ui-fixes-2024-03 into master 7f56c90d97 
Reviewed-on: trent_larson/crowd-funder-for-time-pwa#111
Merge pull request 'send a time for notifications to the push server' (#112) from notify-time into master 8f4289c14d 
Reviewed-on: trent_larson/crowd-funder-for-time-pwa#112
filter by selections (now all working), add cache for plans 640d273646
show in description when recipient is a project (not just Anonymous) 0632fb9b39
Merge branch 'master' into ui-additions-2024-03 5a2a8659f7
Merge pull request 'ui-additions-2024-03' (#113) from ui-additions-2024-03 into master 1a80bbb714 
Reviewed-on: trent_larson/crowd-funder-for-time-pwa#113
on home page, change the filtered button color d49bf61524
add missing Dexie import (which causes failure upon download click) 0fbd1ad51a
reorder home page vapid check to avoid an error on localhost 2ae165d56f
avoid a huge error message in a likely-well-known scenario 78fab735e6
Merge remote-tracking branch 'original-origin/master' into feat/vitejs-trent 5828a290c7
trentlarson added 1 commit 2024-04-07 17:02:26 +00:00
fix linting ab81648aca
trentlarson commented 2024-04-07 17:06:23 +00:00
Author
Contributor
Copy Link

OK, I added some linting fixes.

Note that I changed the default to report lint errors and not fix them, with a separate command to fix them. (I don't mind making "fix" the default as long as we have some command that only reports.)

OK, I added some linting fixes. Note that I changed the default to report lint errors and not fix them, with a separate command to fix them. (I don't mind making "fix" the default as long as we have some command that only reports.)
trentlarson added 1 commit 2024-04-07 17:21:49 +00:00
add linting before any build 28f72640d7
trentlarson added 1 commit 2024-04-07 17:32:04 +00:00
fix title of the test app 5994365a6c
trentlarson added 1 commit 2024-04-07 17:39:07 +00:00
remove a lingering debug console.log 1459719a47
jsnbuchanan merged commit 333ac773f6 into feat/vitejs 2024-04-07 17:52:43 +00:00
trentlarson commented 2024-04-07 18:01:09 +00:00
Author
Contributor
Copy Link

Houston, we have a problem: the vite.config.js is not fully applied on npm run build: the Help page doesn't show a git hash and the sw_scripts-combined.js file is not created and copied into the dist directory.

Houston, we have a problem: the vite.config.js is not fully applied on `npm run build`: the Help page doesn't show a git hash and the sw_scripts-combined.js file is not created and copied into the dist directory.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
jsnbuchanan
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jsnbuchanan/crowd-funder-for-time-pwa#1
Delete Branch "trentlarson/crowd-funder-from-jason:feat/vitejs-trent"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?