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.

Sign up for GitHub

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

Jump to bottom

feat: implement db CRUD operations on backend #11

Merged
merged 19 commits into from
Oct 10, 2023
Merged

feat: implement db CRUD operations on backend #11

merged 19 commits into from
Oct 10, 2023

Conversation

Abhijna-Raghavendra
Copy link
Collaborator

No description provided.

@Abhijna-Raghavendra
feat: add user db for auth
cf0cadf
@Abhijna-Raghavendra
chore: indent frontend code
938b993
@Abhijna-Raghavendra
feat: add vue router
4260d59
@Abhijna-Raghavendra
feat: add sessions to backend
6e7863b
@Abhijna-Raghavendra
refactor: change auth flow
96d1760
@Abhijna-Raghavendra
feat: add router guard
b4a447b
@Abhijna-Raghavendra
fix: implement guard with local storage
3c304f3
@Abhijna-Raghavendra
fix: remove .vite and deno.lock from git
845f99e 
feat: add table to frontend
@Abhijna-Raghavendra
feat: add skeleton home page frontend
c24d695
@Abhijna-Raghavendra
refactor: refactor github user checking; feat: add get maps backend
5f69a92
@Abhijna-Raghavendra
feat: integrate frontend with backend get maps
b27e149
@Abhijna-Raghavendra
feat: create add maps backend
29e8e85
@Abhijna-Raghavendra
fix: fix add maps filter ; fix: fix vite env file
27c8bf9
@Abhijna-Raghavendra
feat: create delete maps backend
10adf28
@Abhijna-Raghavendra
fix: change radio button to dropdown
0461472
@Abhijna-Raghavendra
docs: add sample env
0942c04
@Abhijna-Raghavendra
refactor: add backend address to env
f45eb2e
@Abhijna-Raghavendra
fix: fix auth and homescreen display
704fc41
@Abhijna-Raghavendra
feat: implement jwt tokens
475ddc7
@Abhijna-Raghavendra
Copy link
Collaborator Author

solves #3

BURG3R5
BURG3R5 approved these changes Oct 9, 2023
View reviewed changes
Copy link
Member

@BURG3R5 BURG3R5 left a comment

Choose a reason for hiding this comment

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

All looks good to me. Just check whether you can simplify the db.ts code by using some form of ORM instead of manually making functions for everything. Isn't mongoose an ORM for MongoDB?

@Abhijna-Raghavendra
Copy link
Collaborator Author

seems like mongoose isn't very compatible with deno [https://github.com/Automattic/mongoose/issues/9056]. Will merge PR for now... and research on alternatives soon.

@Abhijna-Raghavendra Abhijna-Raghavendra merged commit f29347d into master Oct 10, 2023
@Abhijna-Raghavendra Abhijna-Raghavendra deleted the orange/db branch October 10, 2023 17:43
@Abhijna-Raghavendra Abhijna-Raghavendra mentioned this pull request Oct 10, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BURG3R5 BURG3R5 BURG3R5 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Abhijna-Raghavendra @BURG3R5