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.

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

Laravel 10 and PHP 8.2 #8277

Open
wants to merge 7 commits into
base: development
Choose a base branch
from
Open

Laravel 10 and PHP 8.2 #8277

wants to merge 7 commits into from

Conversation

sandesh556
Copy link

@sandesh556 sandesh556 commented Dec 20, 2023

This PR ensures that Faveo community version enforces Laravel 10 and PHP 8.2 in the upcoming version. Also all the packages and dependencies have been updated to support the above config

@sandesh556 sandesh556 requested a review from SarfrazS December 20, 2023 07:31
@sandesh556 sandesh556 changed the title laravel 10 Laravel 10 and PHP 8.2 Dec 20, 2023
@bhanu2217 bhanu2217 added this to the v2.0.4 milestone Dec 20, 2023
@bhanu2217 bhanu2217 linked an issue Dec 20, 2023 that may be closed by this pull request
@bhanu2217 bhanu2217 added the PD: Demo Approved Demo of the feature / bug fix was approved by Product team label Dec 20, 2023
@AbishekChinnaraju AbishekChinnaraju self-requested a review January 25, 2024 10:30
@AbishekChinnaraju AbishekChinnaraju added the QA: Round 1 Testing In Progress First round of testing is in progress label Jan 25, 2024
@ganthimathin ganthimathin added the QA: Round 2 Testing In Progress Second round of testing in progress label Jan 25, 2024
@bhanu2217
Copy link
Contributor

With this Faveo Helpdesk should work on both PHP 8.1 and PHP 8.2

@AbishekChinnaraju
Copy link

AbishekChinnaraju commented Jan 29, 2024

Round one testing summary:
Testing time : 1.5 days
Total number of issues: 27
Major:7
Minor:20
Critical:0
Is the pr breaking any functionalities? : NO
Number of times went for functionality correction: 1
Issue sheet: https://docs.google.com/spreadsheets/d/117ZxJfGpsXVd7Kk5adDJis8cVPWhj28IvszHdbOv6ro/edit#gid=0
Note :
We can't able to run composer on the server, it will throw an error.
We can't ablo to test the social logins in local.
In few modules shows 500 errors, so we can't test those modules.

@DhanyaChengappa
Copy link
Contributor

@KNaveenraj-ladybird What is the pending effort required here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Functionality Correction PD: Demo Approved Demo of the feature / bug fix was approved by Product team QA: Round 1 Testing In Progress First round of testing is in progress QA: Round 2 Testing In Progress Second round of testing in progress
Projects
Status: Review in progress
Development

Successfully merging this pull request may close these issues.

Update Laravel & all the libraries
7 participants