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: Migration to Next and Aggregates #80

Merged
merged 41 commits into from
Jul 6, 2022
Merged

feat: Migration to Next and Aggregates #80

merged 41 commits into from
Jul 6, 2022

Conversation

RezaRahemtola
Copy link
Sponsor Member

Description

Project architecture switched to Next, post messages replaced with aggregates and code cleanup.
Cypress tests enhanced.

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes

All users and their content was deleted due to the message type change

Checklist

  • I have assigned this PR to myself
  • I have added at least 1 reviewer
  • I have added the needed labels
  • I have linked this PR to an issue
  • I have linked this PR to a milestone
  • I have linked this PR to a project
  • I have tested this code
  • I have added / updated tests (unit / functionals / end-to-end / ...)
  • I have updated the README and other relevant documents (guides...)
  • I have added sufficient documentation both in code, as well as in the READMEs

RezaRahemtola and others added 30 commits June 24, 2022 14:28
@RezaRahemtola
fix: states set updated with simpler type
9ea2bc1
@RezaRahemtola
feat: temporary migration
a5a6ca2
@RezaRahemtola
feat: Home, Signup and Login pages
2a5265b
@RezaRahemtola
fix: eslint with next
e826c63
@RezaRahemtola
fix: eslint package versions
58589ad
@RezaRahemtola
fix: environement variables using next
4cfe404
@RezaRahemtola
feat: Dashboard page with next
a69447f
@RezaRahemtola
fix: type imports
1ca7a34
@RezaRahemtola
fix: ESLint working with Next config
612cb6c
@RezaRahemtola
feat: ESLint airbnb config updated to latest version
2f2e2a6
@RezaRahemtola
fix: relative imports removed
1239951
@RezaRahemtola
feat: Cypress updated to latest version and with .ts files (#27)
735dd13
@RezaRahemtola
fix: Auth component used in Home page
4db7353
@RezaRahemtola
fix: typos and wrong GitHub URLs in README
3097ab4
@RezaRahemtola
feat: advanced SEO header
a133ad4
@RezaRahemtola
fix: download moved to file card
9e604bc
@RezaRahemtola
feat: default property set to Chakra toast
97059f9
@RezaRahemtola
fix: Cypress GitHub Action updated to latest version
85f8fb7
@RezaRahemtola
fix: Docker build ESLint warning
e908b23
@RezaRahemtola
fix: references to view removed
47a714b
@RezaRahemtola
fix: Empty file upload handled related to #25, Cypress tests added
2235d34
@RezaRahemtola
tests: Cypress hierarchy refactored
8229c36
fix: adapt Dockerfile to next output
6daebb1
feat: better nginx configuration with 404
a6db375
@RezaRahemtola
fix: .prettierrc removed from .dockerignore
36c37ea 
avoid linting warnings when running docker build
@RezaRahemtola
[MERGE] Migration to Next
11b8e31
work in progress: moving computing to aggregates
da71cc2
@RezaRahemtola
feat: Post messages replaced with Aggregate
150493b
@RezaRahemtola
fix: Support for current users added
4b1957c 
Aggregate message key was created only on signup, now on signup and login if needed
@RezaRahemtola
fix: Refactoring with new types and simple toasts
58f8071 
New aggregate types, useToast default params and useless async removed
@RezaRahemtola RezaRahemtola added enhancement New feature or request dependencies Pull requests that update a dependency file labels Jul 6, 2022
@netlify
Copy link

netlify bot commented Jul 6, 2022 •
edited

✅ Deploy Preview for nimble-praline-605cf6 ready!

Name Link
🔨 Latest commit 3fb3143
🔍 Latest deploy log https://app.netlify.com/sites/nimble-praline-605cf6/deploys/62c5f10bb4f6d4000972d480
😎 Deploy Preview https://deploy-preview-80--nimble-praline-605cf6.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 site settings.

adrienfort
adrienfort approved these changes Jul 6, 2022
View reviewed changes
Copy link
Member

@adrienfort adrienfort left a comment

Choose a reason for hiding this comment

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

LGTM

@RezaRahemtola RezaRahemtola merged commit d87237d into master Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adrienfort adrienfort adrienfort approved these changes

Assignees

@ZerLock ZerLock

@RezaRahemtola RezaRahemtola

@AmozPay AmozPay

@Alex-Prevot Alex-Prevot

@MizuriGit MizuriGit

@Samoten777 Samoten777

Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@RezaRahemtola @adrienfort @ZerLock @AmozPay @Alex-Prevot @MizuriGit @Samoten777