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

Implemented: Static UI for the more jobs sections(#2wn5yyv) #249

Closed
wants to merge 3 commits into from

Conversation

shashwatbangar
Copy link
Contributor

Related Issues

Closes #

Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to add changelog entry

Contribution and Currently Important Rules Acceptance

Copy link
Contributor

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

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

small changes, otherwise looks good.

src/locales/en.json Outdated Show resolved Hide resolved
src/views/Inventory.vue Outdated Show resolved Hide resolved
@shashwatbangar shashwatbangar changed the title Implemented: Static UI for the more jobs sections(#wn5yyv) Implemented: Static UI for the more jobs sections(#2wn5yyv) Oct 17, 2022

<ion-card>
<ion-card-header>
<ion-card-title>{{ $t("More jobs") }}</ion-card-title>
Copy link
Contributor

Choose a reason for hiding this comment

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

Add this to en.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done sir

@@ -33,6 +33,26 @@
</ion-label>
</ion-item>
</ion-card>

<ion-card>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should create a separate component and reuse throughout the app

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes sir. We do have a separate component called "MoreJobs" but I created it in the PR I've added functionality part.
PR - #250
Do you want me to display static data through a separate component "MoreJobs"?

@adityasharma7
Copy link
Contributor

This is done with #274

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.

3 participants