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

Release refactor changes to main #247

Merged
merged 191 commits into from
Aug 24, 2024
Merged

Release refactor changes to main #247

merged 191 commits into from
Aug 24, 2024

Conversation

tomaspalma
Copy link
Member

A feature affected from the refactor that is not being sent to main is the random fill but it will be sent soon enough

Wallen-Ribeiro and others added 30 commits October 16, 2023 18:09
@Wallen-Ribeiro
Reduced text in 'intro' and 'why' section
b140273
@Wallen-Ribeiro
Added banner to About page
21b8dd7
@Wallen-Ribeiro
cleaned updates info
2f2e37c
@Wallen-Ribeiro
updated home-page imgs
d32312a
@LuisDuarte1
Merge pull request #187 from NIAEFEUP/fix/check-outdated-data
32f1147 
Fixed info request location
@diogotvf7
Fixed endpoint path
28e715f
@LuisDuarte1
Merge pull request #189 from NIAEFEUP/fix/check-outdated-data
6e16578 
Fixed endpoint path
@Wallen-Ribeiro
added ni-stamp v1
7789d32
@thePeras
Remove outdated faqs
8586c7a
@diogotvf7
Removed unnecessary console.logs
3d7c657
@diogotvf7
Bug on paste fixed!
e17b446
@thePeras
Add new FAQS and modify old ones
ba22856
@thePeras
Faq default close
f980d50
@diogotvf7
Fixed randomizer | Locked button not switching state before switching…
30ea8cc 
… option needs fix
@LuisDuarte1
Make an action to wait for netlify deploy
77c9d9f
@jpnsantoss
preview config
6d9430f
@jpnsantoss
added .env.example
36183ed
@jpnsantoss
Revert "added .env.example"
714d797 
This reverts commit 36183ed.
@jpnsantoss
Revert "preview config"
2def73a 
This reverts commit 6d9430f.
@jpnsantoss
revert changes + netlify spa
3bd1771
@LuisDuarte1
Merge pull request #193 from NIAEFEUP/devops/wait-for-netlify
6d28b52 
Make an action to wait for netlify deploy for develop preview
@LuisDuarte1
Merge pull request #194 from NIAEFEUP/feat/preview
5c1919a 
Preview Config
@jpnsantoss
New dropdown menu
14c1c4f
@jpnsantoss
Renamed component
c2be415
@diogotvf7
Added tooltip on buttons
e9927fc
@diogotvf7
Fixed small bug
41c7911
@diogotvf7
Fixed print on dark mode
d972ac2
@jpnsantoss
dark mode
9ae207a
@diogotvf7
Added tooltips to buttons
33857e1
@diogotvf7
Updated courses order to mimic sigarra
1794fc3
@tomaspalma tomaspalma requested a review from a team August 14, 2024 22:39
@netlify Netlify
Copy link

netlify bot commented Aug 14, 2024 •
edited
Loading

✅ Deploy Preview for tts-fe-preview ready!

Name Link
🔨 Latest commit 6ea60dc
🔍 Latest deploy log https://app.netlify.com/sites/tts-fe-preview/deploys/66ca2e0efd4a7100083bd645
😎 Deploy Preview https://deploy-preview-247--tts-fe-preview.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 configuration.

@tomaspalma tomaspalma changed the title Merge into develop Merge into main Aug 14, 2024
@tomaspalma tomaspalma changed the title Merge into main Release refactor changes to main Aug 14, 2024
thePeras
thePeras approved these changes Aug 22, 2024
View reviewed changes
Copy link
Member

@thePeras thePeras left a comment

Choose a reason for hiding this comment

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

+8,998 −32,776 Indeed, an intense refactor

✅ 🛫

jose-carlos-sousa
jose-carlos-sousa approved these changes Aug 22, 2024
View reviewed changes
Copy link
Contributor

@jose-carlos-sousa jose-carlos-sousa left a comment

Choose a reason for hiding this comment

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

LGTM

Process-ing
Process-ing requested changes Aug 23, 2024
View reviewed changes
Copy link
Contributor

@Process-ing Process-ing left a comment

Choose a reason for hiding this comment

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

It's best to merge #254 first

@tomaspalma tomaspalma merged commit f74de38 into main Aug 24, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Process-ing Process-ing Process-ing approved these changes

@thePeras thePeras thePeras approved these changes

@jose-carlos-sousa jose-carlos-sousa jose-carlos-sousa approved these changes

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

Successfully merging this pull request may close these issues.