CVS edit link for matixbg

At the Drupal Meetup in Sofia (17.04.2010) the community expressed that there is a need of "ePay.bg" payment module for Ubercart 6. Currently only a version of the module for the 5.x branch exists. I've contacted the uc_epaybg module maintainer and he answered he'll be happy if I'm a co-maintainer: http://drupal.org/node/773850#comment-3010362
The module rewrite is ready and I would like to release a 6.x.1.x-dev version so people can test it.
Please advise if I have to provide something more.
Thank you!
CommentFileSizeAuthor
#1 uc_epaybg.tgz7.95 KBmmartinov
#2 uc_epaybg.tgz7.94 KBmmartinov
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mmartinov’s picture

FileSize
7.95 KB

I'm adding the new version of the module for review.

* Edit: Please use the file from the next comment.

mmartinov’s picture

Status: Postponed (maintainer needs more info) » Needs review
FileSize
7.94 KB

Fixed a wrong variable name, reported by geobaev.

apaderno’s picture

Status: Needs review » Fixed

Thank you for your contribution! I am going to update your account.

These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.

mmartinov’s picture

Status: Fixed » Needs review

Hey, thanks for the CVS access. Is there anything else I should do in order to be able to commit the new version of the module?
Currently, when trying to commit in the DRUPAL-6--1 branch I get the following error:

Access denied: matixbg does not have permission to commit files in:
contributions/modules/uc_epaybg/uc_epaybg

Also at the module page I don't have the "Add new release" and "Administer releases" links under "View all releases".

apaderno’s picture

Status: Needs review » Fixed

The current maintainer should have added you to the list of the users with permission to access the project repository; I added you, and you are now granted CVS access to the repository.

martichka’s picture

Hello,
I have installed the new version, and when I press Review order I get the following message:

* user warning: Unknown column 'txn_id' in 'field list' query: SELECT COUNT(txn_id) FROM uc_epaybg_transaction WHERE txn_id = 52 in /var/www/biogradina.com/sites/all/modules/uc_epaybg/uc_epaybg.module on line 201.

* user warning: Unknown column 'txn_id' in 'field list' query: INSERT INTO uc_epaybg_transaction (txn_id, description) VALUES (52, 'a:1:{s:6:\"amount\";d:23;}') in /var/www/biogradina.com/sites/all/modules/uc_epaybg/uc_epaybg.module on line 214.

Btw, many thanks for the initiative and the work! :)

mmartinov’s picture

Hi, please use the project issue tracker at http://drupal.org/project/issues/uc_epaybg for such reports. I'll make a release on the project page soon.

mmartinov’s picture

Status: Fixed » Closed (fixed)
apaderno’s picture

Component: Miscellaneous » co-maintainer application
Assigned: Unassigned » apaderno
Issue summary: View changes