Personal deadlines #300
Personal deadlines #300
Conversation
| } | ||
| } | ||
|
|
||
| func timedateStringFromDate(date: Date) -> String { |
There was a problem hiding this comment.
Ну, у нас есть ещё несколько мест, где мы Date форматируем как-то.
Stepic/PersonalDeadlineCounter.swift
Outdated
| } | ||
| fulfill((section.id, sectionTimeToComplete)) | ||
| }, error: { | ||
| //TODO: Add error handling here |
There was a problem hiding this comment.
Нужно добавить хотя бы логирование, я считаю.
There was a problem hiding this comment.
да, упустил эту тудушку вообще
| private func countTimeToComplete(section: Section) -> Promise<(Int, TimeInterval)> { | ||
| return Promise { | ||
| fulfill, _ in | ||
| section.loadUnits(success: { |
There was a problem hiding this comment.
Если это возможно, стоит избавляться от коллбеков сразу.
There was a problem hiding this comment.
Там легаси, которое надо рефакторить, когда будет переделывать старые классы модели типа Course, Section, Lesson
| import UIKit | ||
| import ActionSheetPicker_3_0 | ||
|
|
||
| class PersonalDeadlineEditScheduleViewController: UIViewController { |
There was a problem hiding this comment.
Это не MVP модуль? Кажется, что здесь совсем легко можно разделить логику.
| extension PersonalDeadlineEditScheduleViewController: UITableViewDelegate { | ||
| func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) { | ||
| AnalyticsReporter.reportEvent(AnalyticsEvents.PersonalDeadlines.EditSchedule.Time.opened) | ||
| ActionSheetDatePicker.show(withTitle: NSLocalizedString("SelectTimeTitle", comment: ""), datePickerMode: UIDatePickerMode.dateAndTime, selectedDate: sectionDeadlinesData[indexPath.row].deadline, minimumDate: Date(), maximumDate: Date().addingTimeInterval(60 * 60 * 24 * 30 * 365), doneBlock: { |
There was a problem hiding this comment.
Возможно, произведения единиц времени стоит выносить в константы, чтобы было понятно, что это за интервал.
| collectionView.constrainHeight("\(modeButtonSize.height)") | ||
| collectionView.invalidateIntrinsicContentSize() | ||
| collectionView.layoutIfNeeded() | ||
| (collectionView.collectionViewLayout as? UICollectionViewFlowLayout)?.itemSize = modeButtonSize |
There was a problem hiding this comment.
Ниже дублирование кода. Возможно стоит сделать свой лэйаут или хотя бы в отдельный метод.
| import FLKAutoLayout | ||
| import SVProgressHUD | ||
|
|
||
| class PersonalDeadlinesModeSelectionViewController: UIViewController { |
There was a problem hiding this comment.
Тут тоже про MVP вопрос. Правда, здесь ещё меньше логики, которую можно извлечь.
| override func setupSubviews() { | ||
| self.view.setRoundedCorners(cornerRadius: 8) | ||
| yesButton.setRoundedCorners(cornerRadius: 8, borderWidth: 1, borderColor: UIColor(hex: 0x45B0FF)) | ||
| //TODO: Do not forget to localize text and buttons here |
Stepic/StepikLabel.swift
Outdated
| case .gray: | ||
| return UIColor.lightGray | ||
| case .blue: | ||
| return UIColor(hex: 0x45B0FF) |
There was a problem hiding this comment.
В трёх местах уже используется. Вынесем?
There was a problem hiding this comment.
Что вынесем? Цвет? Давай.
Stepic/StorageData.swift
Outdated
|
|
||
| protocol StorageData { | ||
| init(json: JSON) | ||
| var dictValue: [String : Any] { |
There was a problem hiding this comment.
Всегда ж было { get set } в одну строку :)
Задача: #APPS-1880
Коротко для Release Notes, в формате «Сделали/Добавили/Исправили N»:
Добавили пользовательские дедлайны
Описание:
Основные классы:
PersonalDeadlineCounter- класс, ответственный за рассчет времениPersonalDeadlineLocalStorageManager- класс, ответственный за хранениеStorageRecordобъектовPersonalDeadlineNotificationsManager- класс, ответственный за установку нотификаций по дедлайнамStorageRecordsAPI- АПИ класс работы со эндпоинтомapi/storage-recordsPersonalDeadlineManager- класс, ответственный за реализацию всех кейсов использования персональных дедлайновPersonalDeadlineEditScheduleViewController- контроллер изменения расписанияPersonalDeadlineModesSelectionViewController- контроллер выбора режимов изученияСуществует несколько кейсов для использования пользовательских дедлайнов, каждый из которых разрешается методом в
PersonalDeadlineManager:func countDeadlines(for course: Course, mode: DeadlineMode)PersonalDeadlineCounterStorageRecordsAPIPersonalDeadlineLocalStorageManagerPersonalDeadlineNotificationsManagersyncDeadline(for course: Course, userID: Int)StorageRecordsAPIпо kind с ID нужного нам курсаPersonalDeadlineLocalStorageManagerлокальную запись для курсаPersonalDeadlineNotificationsManagerfunc deleteDeadline(for course: Course)PersonalDeadlineLocalStorageManagerStorageRecordsAPIс полученным ID записиPersonalDeadlineLocalStorageManagerлокальную запись для курсаPersonalDeadlineNotificationsManagerchangeDeadline(for course: Course, newDeadlines: [SectionDeadline])PersonalDeadlineLocalStorageManagerStorageRecordsAPIс подмененными дедлайнами в полученном объектеPersonalDeadlineLocalStorageManagerлокальную запись для курсаPersonalDeadlineNotificationsManagerДля установки дедлайнов в
PersonalDeadlineNotificationsManagerиспользуется фреймворкUserNotifications. Подсчет времени вPersonalDeadlineCounterпроисходит по полюtime_to_completeдля каждого урока. Core Data была смигрирована на новую версию для храненияtime_to_complete. StorageRecord объекты хранятся в UserDefaults.