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

Jobs List Screen Design #184

Merged
merged 8 commits into from
Jun 18, 2024
Merged

Conversation

tobitech
Copy link
Contributor

Story: https://app.shortcut.com/smileid/story/12793

Summary

This includes the following changes:

  • Design implementation of Jobs List View and Job List Item
  • Add Jobs View to the app's main TabView
  • Design implementation of EmptyState View component
  • Extend SmileID JobType to have label and icon properties.
  • Move reusable views into components group.

Known Issues

N/A

Test Instructions

Launch the demo app and you should see a new Jobs tab added to the tab view.

Screenshot

Jobs List Empty State

@tobitech tobitech self-assigned this Jun 17, 2024
@tobitech tobitech requested a review from a team as a code owner June 17, 2024 15:57
Copy link

github-actions bot commented Jun 17, 2024

Warnings
⚠️ The source files were changed, but the tests remain unmodified. Consider updating or adding to the tests to match the source changes.

Generated by 🚫 Danger Swift against 7b893a3

Copy link
Contributor

@vanshg vanshg left a comment

Choose a reason for hiding this comment

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

  • Can you make it so that the JobItem expands when you click on any part of it, not just the arrow?
  • Can you make the user id/job id selectable/copyable

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for reorganizing this

Comment on lines +61 to +62
#if DEBUG
extension JobData {
Copy link
Contributor

Choose a reason for hiding this comment

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

This is great

@tobitech tobitech requested a review from vanshg June 18, 2024 10:37
Copy link
Contributor

@vanshg vanshg left a comment

Choose a reason for hiding this comment

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

This looks great!

@tobitech tobitech merged commit 4b4de90 into jobs-list-screen Jun 18, 2024
3 checks passed
@tobitech tobitech deleted the task-jobs-list-screen-design branch June 18, 2024 17:43
@tobitech tobitech mentioned this pull request Jul 15, 2024
tobitech added a commit that referenced this pull request Jul 16, 2024
* Jobs List Screen Design (#184)

* change folder structure. add smileid theme background to the jobs screen.

* add icon computed property to the job type

* design implementation for empty state view

* move reusable views into components group

* add background and spacers to empty state view

* code formatting

* make user id and job id copyable.

* expand job item when container is tapped

* Task Core Data Setup (#185)

* change folder structure. add smileid theme background to the jobs screen.

* add icon computed property to the job type

* design implementation for empty state view

* move reusable views into components group

* some project restructuring

* add core data model file to the project. define an entity for job and add configure its attributes. generate model class

* setup core data stack with persistent container and save and delete model object functionalities.

* add an error model

* setup core data stack with preview version. add create function to the job managed object.

* implement new static functions for fetching jobs and saving new jobs.

* add update and delete jobs static functions

* code formatting.

* model target changes.

* change path of xcdatamodel file in pbxproj file.

* move model file back to models group.

* remove hard coded path

* Task Save and Fetch Jobs (#186)

* add a view model for jobs view and a provider to handle data store.

* add a confirmation to clear jobs action.

* save information from successful jobs in the corresponding delegate methods.

* some code formatting

* specify todos - to be removed after clarification.

* move the creation of user id and job id to HomeViewModel and pass the values into the different jobs screen.

* code formatting.

* update job complete for smart selfie authentication

* Task Update Jobs Status (#192)

* improve error handling for functionalities using the data store. introduce new model to handle updating each job status.

* update all the necessary details of a job stored locally from the api response.

* code formatting

* update job element model

* Sort Jobs List and Fix Job Status Update Issue (#194)

* change job timestamp to date type. add a sort descriptor to jobs fetch request.

* make job timestamp a non-optional date. add errorDescription to data store error model. correct attribute key for sortdescriptor.

* inject userId and jobId from home view model into the verification product flows.

* add a new case for ActionResult.

* rename sortdescriptor

* Fix TabBar Background (#196)

* restore ipad support and disable ipad multitasking

* fix tab bar background issue caused by toast view.

* Fix Jobs User Id issue (#197)

* use newly generated userid for authentication jobs. pass in userId and jobId into authentication request.

* bump build number.

* hide clear jobs button if there are no jobs

* Status Progress and Scoping of Persisted Jobs (#198)

* add an indeterminate linear progress view to job list item and show it when fetching job status.

* add isProduction and partnerId attributes to Job model.

* inject config into the JobsViewModel to gain access to partnerId

* code formatting.

* add a new ActionResult not previously added.
hide progress view when get status task is canceled

* add an unknown case to ActionResult. Display partnerId and environment on JobListeItem

* add doc to newly introduced unknown case.

* shorten the doc text for unknown case

* Refresh Entire App when Smile Config is Updated (#199)

* move smile config decoding and initialisation into root view model. add a closure to send back update config to root view.

* update init method

* bump build number. prevent joblistitem preview from running in release environment.

* update changelog
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