Wrong column in Pop3 Connector view table

dz00te · September 16, 2015, 10:08am

Continuing the discussion from New column in pop3 connector table:

now:

+----------------+---------------------+
|    Account     |       Actions       |
+----------------+---------------------+
| email addressA | Disable/Edit/Delete |
| email addressB | Disable/Edit/Delete |
| email addressC | Disable/Edit/Delete |
+----------------+---------------------+

to something like:

+---------+---------------------+------------+-----+----------+--------+---------+
| Account |       Actions       | Local User | SSL | Protocol | Delete | Enabled |
+---------+---------------------+------------+-----+----------+--------+---------+
| email A | Disable/Edit/Delete | user1      | Yes | pop3     | yes    | no      |
| email B | Disable/Edit/Delete | user2      | No  | imap     | yes    | yes     |
| email C | Disable/Edit/Delete | user1      | Yes | imap     | no     | no      |
+---------+---------------------+------------+-----+----------+--------+---------+

or maybe better (if Disabled we can see in actions column):

+----------------+------------+-----+----------+--------+---------------------+
|    Account     | Local User | SSL | Protocol | Delete |       Actions       |
+----------------+------------+-----+----------+--------+---------------------+
| email addressA | user1      | Yes | pop3     | yes    | Disable/Edit/Delete |
| email addressB | user2      | No  | imap     | yes    | Disable/Edit/Delete |
| email addressC | user1      | Yes | imap     | no     | Disable/Edit/Delete |
+----------------+------------+-----+----------+--------+---------------------+

what do you think?

update: to get the idea of what i mean

i think i’ve made an error, sorry, the 2nd column was to be ‘account’ and not ‘username’

il’’ try to correct soon

alefattorini · September 16, 2015, 10:38am

Thanks for reporting it! Do you wanna try also with a Pull Request?

dz00te · September 16, 2015, 11:08am

Yes, of course.

giacomo · September 16, 2015, 3:24pm

The package is already inside the main mirror ready for QA (http://mirror.nethserver.org/nethserver/6.6/testing/x86_64/Packages/nethserver-fetchmail-1.1.3-1.5.g9d2a593.ns6.noarch.rpm).

Please @dz00te let someone else do the QA (Probably I will do it tomorrow)

dz00te · September 16, 2015, 4:02pm

wow, tnx for the ultra-fast merge

to testing team @medworthy, @GG_jr, @vcc, @fasttech, @mabeleira
if someone has time it could test it in place of @giacomo?
so he can commit those 10/15 minutes to develop new features?

GG_jr · September 16, 2015, 4:29pm

This is?

mabeleira · September 16, 2015, 5:57pm

INstalled updates too

alefattorini · September 16, 2015, 7:55pm

That’s a great team! Dream team

alefattorini · September 17, 2015, 9:06am

@mabeleira @GG_jr please use this docs to do the QA correctly

dz00te · September 17, 2015, 10:01am

@GG_jr @mabeleira tnx for quick tests
in addition to links posted by @alefattorini , hoping to have soon an howto/guide, see also the devs manual:
strictly related to our group:
http://docs.nethserver.org/projects/nethserver-devel/en/latest/development_process.html#qa-team-member

The QA team member

Takes an unassigned issue ON_QA state and sets the Assignee field to herself.

Tests the package, following the test case documentation written by the Developer

She can set NEEDINFO flag if informations about how to test the code are missing.

When test is passed she sets the issue state to VERIFIED, otherwise she puts it back in TRIAGED state cleaning the Assignee field.

or if you have time… useful to understand all the development process:
http://docs.nethserver.org/projects/nethserver-devel/en/latest/development_process.html