Skip to content
  • Projects
  • Groups
  • Snippets
  • Help

Codecrew / Moya

  • This project
    • Loading...
  • Sign in
Go to a project
  • Project
  • Repository
  • Issues 30
  • Merge Requests 2
  • Wiki
  • Snippets
  • Settings
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Commits
  • Issue Boards
Merged
Merge Request !178 opened Dec 16, 2014 by Tuomas Riihimäki@tuomari

Updates

Look at the commits

Edited Dec 16, 2014
Request to merge tuomari:releaseplace into master

Merged

  • The changes were merged into master.
  • The source branch has been removed.
  • Discussion 1
  • Commits 13
  • Changes 22
  • {{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • 2a54e04ac87f85d4d091c3be1c8f9cea?s=80&d=identicon
    Tuukka Kivilahti
    @tkfftk started a discussion on the diff Dec 16, 2014
    code/moya-beans/ejbModule/fi/codecrew/moya/beans/PlaceBean.java
    618 618 // remove also slot from place
    619 619 if (slot != null) {
    620 620 slot.setPlace(null);
    621 slot.setUsed(null);
    • Tuukka Kivilahti @tkfftk commented Dec 16, 2014
      Owner

      Jätin alunperin tarkoituksella pois. Koska normaalisti workflow asiakkaan siirtoon on että admin eka varaa asiakkaalle uuden paikan ja sitten vapauttaa vanhan (näin estetään ettei uus paikka kerkeä mennä tässä välissä ns. nenän alta) -> tällä muutoksella asiakkaalle jää yksi käyttämätön placeslot jonka se voi käyttää mihin tahansa.

      Itseasiassa, pitäisikö tästä tehdä optio, jotta admin voi päättää että vapauttaako se tämän uudelleenvalittavaksi vai miten?

  • Tuomas Riihimäki @tuomari

    Added 2 new commits:

    • 2fbbded3 - IRC server password
    • b4ee141e - Bill ID to list
    Dec 16, 2014

    Added 2 new commits:

    • 2fbbded3 - IRC server password
    • b4ee141e - Bill ID to list
    Toggle commit list
  • Tuukka Kivilahti @tkfftk

    mentioned in commit 39f25ee8

    Dec 16, 2014

    mentioned in commit 39f25ee8

    Toggle commit list
  • Write
  • Preview
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
  • Please register or sign in to post a comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View labels
2
2 participants
Reference: codecrew/moya!178