Skip to content

Deploy#51

Merged
arecouz merged 19 commits intomainfrom
deploy
Jun 14, 2025
Merged

Deploy#51
arecouz merged 19 commits intomainfrom
deploy

Conversation

@Jaz-spec
Copy link
Contributor

✨ What’s Changed?

  • Front end now calls deployed backend, not local host
  • pulled in saveUserEvents

Oggie112 and others added 18 commits June 9, 2025 12:22
… the event to their user page and to their profile in the DB. For non-users it takes them to the login/signup page.

This  change required a fair bit of refactoring of a number of different parts of code. The events / users api's now use axios. The eventsPage now contains more logic for the save and displaying of user events and the subsequent components pass more of this state/props through. Fixed navbar bug by adding a defined bg color. Use events commented out for the time being. Filter now set to all on load rather than just music.
… numbers remain numbers. Forms auto to strings.
@netlify
Copy link

netlify bot commented Jun 14, 2025

Deploy Preview for the-locals ready!

Name Link
🔨 Latest commit aefb64e
🔍 Latest deploy log https://app.netlify.com/projects/the-locals/deploys/684d65296d61cf0008615c6b
😎 Deploy Preview https://deploy-preview-51--the-locals.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@FortyTwoFortyTwo FortyTwoFortyTwo left a comment

Choose a reason for hiding this comment

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

Would be nice to have deployed URL be put in an environment variable without needing to paste it everywhere, it's to also help continue run it locally by changing the env variable.

@arecouz arecouz merged commit d65bbf5 into main Jun 14, 2025
6 checks passed
@arecouz arecouz deleted the deploy branch June 14, 2025 13:30
@arecouz arecouz restored the deploy branch June 14, 2025 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants