Ticket #234 (closed planned_task: obsolete)

Opened 11 years ago

Last modified 10 years ago

BASE_DND_R0

Reported by: Astea Owned by: peko
Priority: 3 Milestone: M04_PRE4
Component: BASE_GUI_LIB Version: 2.0
Keywords: Cc:
Category: BASE Effort: 2
Importance: 0 Ticket_group:
Estimated Number of Hours: Add Hours to Ticket:
Billable?: Total Hours:
Analysis_owners: peko, peko, vlado Design_owners: peko, peko, vlado, peko
Imp._owners: peko Test_owners:
Analysis_reviewers: boyan, dido Changelog:
Design_reviewers: meddle Imp._reviewers: meddle
Test_reviewers: Analysis_score: 3.5
Design_score: 3.5 Imp._score: 3
Test_score: 0

Description

wiki page: BASE_DND_R0 - effort: 2d

Change History

comment:1 Changed 11 years ago by peko

  • Owner changed from Astea to peko
  • Design_score set to 0
  • Imp._score set to 0
  • Test_score set to 0
  • Analysis_score set to 0

comment:2 Changed 11 years ago by peko

  • Status changed from new to s1a_analysis_started
  • Analysis_owners set to peko

comment:3 Changed 11 years ago by peko

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:4 Changed 11 years ago by boyan

  • Status changed from s1b_analysis_finished to s1c_analysis_ok
  • Analysis_reviewers set to boyan
  • Analysis_score changed from 0 to 3.5

The analysis is good - you have provided a lot of useful information but the bullets in the Task requirements section might have been structured better.

comment:5 Changed 11 years ago by peko

  • Design_owners set to peko
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:6 Changed 10 years ago by peko

  • Status changed from s2a_design_started to new
  • Analysis_owners changed from peko to peko, peko, vlado

comment:7 Changed 10 years ago by peko

  • Status changed from new to s1a_analysis_started

comment:8 Changed 10 years ago by peko

  • Status changed from s1a_analysis_started to s1b_analysis_finished

comment:9 Changed 10 years ago by dido

  • Status changed from s1b_analysis_finished to s1c_analysis_ok
  • Analysis_reviewers changed from boyan to boyan, dido

It is fine but as far as I understand it drag and drop should allow import and export of files. In this case for example if user drags out an image to the desktop, before the image is saved as file there should be a dialog that allows the user to choose file name, format and other options(image quality, alpha chanel etc.)
analysis review 3.5p (30m)

comment:10 Changed 10 years ago by vlado

  • Design_owners changed from peko to peko, vlado
  • Owner changed from peko to vlado
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:11 Changed 10 years ago by vlado

  • Status changed from s2a_design_started to s2b_design_finished

comment:12 Changed 10 years ago by vlado

  • Design_owners changed from peko, vlado to peko, peko, vlado

comment:13 Changed 10 years ago by peko

  • Status changed from s2b_design_finished to s1c_analysis_ok

comment:14 Changed 10 years ago by peko

  • Owner changed from vlado to peko
  • Status changed from s1c_analysis_ok to s2a_design_started

comment:15 Changed 10 years ago by peko

  • Design_owners changed from peko, peko, vlado to peko, peko, vlado, peko
  • Status changed from s2a_design_started to s2b_design_finished

comment:16 Changed 10 years ago by meddle

  • Status changed from s2b_design_finished to s2c_design_ok
  • Design_score changed from 0 to 3.5
  • Design_reviewers set to meddle

I think that the design is fine and the documentation is wonderful. The whole idea and the API is not simple, but the tests was of help to understand the basics... There are some code quality issues... but I fixed them. Read GoodCodeExamples.

3.5p (80m)

comment:17 Changed 10 years ago by peko

  • Status changed from s2c_design_ok to s3a_implementation_started
  • Imp._owners set to peko

comment:18 Changed 10 years ago by peko

  • Status changed from s3a_implementation_started to s3b_implementation_finished

comment:19 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Imp._score changed from 0 to 3
  • Imp._reviewers set to meddle

The implementation works, but I had a hard time merging because of your rare synchronization with the trunk. The module didn't run on true author/reader, I fixed that, next time make sure, your module run and build correctly with the true launchers. There were classes not mentioned in the design too, you fixed that...

3p (90m)

comment:20 Changed 10 years ago by deyan

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to obsolete

Batch update from file query-obsoleted.csv

Note: See TracTickets for help on using tickets.