Skip to content

Job Categories Modifid to fix Migrate command#70

Open
MDARH wants to merge 1 commit intotheihasan:mainfrom
MDARH:FixMigrationIssue
Open

Job Categories Modifid to fix Migrate command#70
MDARH wants to merge 1 commit intotheihasan:mainfrom
MDARH:FixMigrationIssue

Conversation

@MDARH
Copy link
Copy Markdown

@MDARH MDARH commented Jun 19, 2025

📝 Why is PR is required?

Answer:

✅ Checklist

  • My code follows the project's contribution guidelines.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added necessary documentation (if applicable).
  • I have tested this code locally.

🛠️ Changes

  • Bug fix (non-breaking change that fixes an issue)
  • Feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactor (code improvement without changing external behavior)
  • Other (please specify):

📌 Related Issues (e.g., Fixes #123, Closes #456)

Ans: Migration

📸 Screenshots (if applicable)

Ans: None

📣 Additional Comments

Ans: Php artisan migration was not working. I have made change only migrations file

Comment thread config/mail.php
'url' => env('MAIL_URL'),
'host' => env('MAIL_HOST', '127.0.0.1'),
'port' => env('MAIL_PORT', 2525),
'port' => env('MAIL_PORT', '2525'),
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why string instead of integer?


Schema::table('job_listings', function (Blueprint $table) {
$table->dropColumn('job_category');
});
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Job category has a relationship with job listings. zso why are you remove this column

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.

2 participants