Skip to content

build: esm migration #577

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
Draft
emmebravo wants to merge 19 commits into
base: main
Choose a base branch
from
Draft

build: esm migration #577

emmebravo wants to merge 19 commits into from

Conversation

@emmebravo
Copy link
Collaborator

@emmebravo emmebravo commented Jul 8, 2025

Description

  • updated mongodb, mongoose, and connect-mongo
  • moved supertest back to dev dependencies as I had moved it to normal dependencies by accident
  • ESM modules update
  • cypress config update to fit ESM modules

Type of change

Please select everything applicable. Please, do not delete any lines.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change requires an update to testing

Issue

Checklist:

  • This PR is up to date with the main branch, and merge conflicts have been resolved
  • I have executed npm run test and npm run test:e2e and all tests have passed successfully or I have included details within my PR on the failure.
  • I have executed npm run lint and resolved any outstanding errors. Most issues can be solved by executing npm run format
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@emmebravo @alcpereira
build: update mongodb setup to fit with esm migration
92a5ea4 
updated mongodb, mongoose, and connect-mongo; moved supertest back to dev dependencies as I had moved it to normal dependencies by accident

closes Together-100Devs#476 and Together-100Devs#493
@alcpereira alcpereira force-pushed the enhancement/issue-476 branch from d7c13ee to 92a5ea4 Compare July 15, 2025 20:03
AngelCodes95 and others added 18 commits July 30, 2025 20:26
@AngelCodes95 @emmebravo
chore: change root package name to lowercase
9cfa66f
@emmebravo
setup fly.io automatic deploys with github actions
368b78c
@alcpereira @emmebravo
Revert "feat: set cookie with 7 day max age"
c5a8ea0 
This reverts commit 4d3c947.
@AngelCodes95 @emmebravo
Fix: Discord OAuth redirect issue
83770b7 
Add trust proxy setting to enable HTTPS detection behidn Fly.io proxy.

Fixes issue Together-100Devs#466
@DevinCLane @emmebravo
add codeowners file
dff9a5d
@AngelCodes95 @emmebravo
feat: restore cookie with 7 day max age
2ba25c5 
This reverts commit d40983b which had reverted the original cookie max age feature.
The cookie max age functionality is needed for proper session management.

Addresses issue Together-100Devs#599
@DevinCLane @emmebravo
Update CODEOWNERS with eric
5844217 
add intelagense to codeowners
@absar22 @emmebravo
fix: capitalize 'D' in all instances of 100Devs Together-100Devs#604
0dc3cbb
@snikmas @emmebravo
fixed a tringy
75b7121
@alcpereira @emmebravo
chore: update packages versions that are vulnerable
a27192e
@merrua @emmebravo
docs: move to the main repo
505c960 
move the help text from the wiki to the docs
@merrua @emmebravo
docs: fix broken links after move
c2126f7
@cevangel @emmebravo
chore: delete setup proxy
f194b5d 
Delete client/src/setupProxy.js

Added title and aria-label attributes to CalendarHeader.jsx

Revert "Added title and aria-label attributes to CalendarHeader.jsx"

This reverts commit 39c52a6.

client/src/features/calendarHeader/CalendarHeader.jsx reverted adding
title/aria-label attributes to calendarHeader.jsx

Remove http-proxy-middleware from client and update codebase

Undo title and aria-label changes for CalendarHeader.jsx
@adel-abdulazeem @emmebravo
fix: typo WelcomUserModel comp Cant => Can't
8f8cbb2
@Aswanth-v @emmebravo
feat(ui): add cursor pointer and hover effects to all buttons
16af6b1
@dan-foley @emmebravo
Edit Discord 0Auth2 Docs
457621e 
Complete requested changes

Edit Discord 0Auth2 Docs

Edit Discord 0Auth2 Docs

Fixed button screenshot

Trying to get image to load correctly

Finish Discord 0Auth2 Setup instructions

Note to check login

Fix redirect button image

Fix localhost link
@Hailo7ts @emmebravo
added contributing guidelines to readme
9bba01d
@emmebravo
build: update mongodb setup to fit with esm migration
933f454 
updated mongodb, mongoose, and connect-mongo; moved supertest back to dev dependencies as I had moved it to normal dependencies by accident

closes Together-100Devs#476 and Together-100Devs#493
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DevinCLane DevinCLane Awaiting requested review from DevinCLane DevinCLane will be requested when the pull request is marked ready for review DevinCLane is a code owner

@alcpereira alcpereira Awaiting requested review from alcpereira alcpereira will be requested when the pull request is marked ready for review alcpereira is a code owner

@moses-codes moses-codes Awaiting requested review from moses-codes moses-codes will be requested when the pull request is marked ready for review moses-codes is a code owner

@funbunch funbunch Awaiting requested review from funbunch funbunch will be requested when the pull request is marked ready for review funbunch is a code owner

@merrua merrua Awaiting requested review from merrua merrua will be requested when the pull request is marked ready for review merrua is a code owner

@intelagense intelagense Awaiting requested review from intelagense intelagense will be requested when the pull request is marked ready for review intelagense is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Migrate server to ESM modules